Skip to content

Include indirect costs in vertical and lateral (non-vertical) section drilling costs [v3.17] - #180

Closed
softwareengineerprogrammer wants to merge 3 commits into
mainfrom
vertical-and-lateral-indirect-costs
Closed

softwareengineerprogrammer wants to merge 3 commits into
mainfrom
vertical-and-lateral-indirect-costs

Conversation

@softwareengineerprogrammer

Copy link
Copy Markdown
Owner

Description

  1. Addresses Drilling and completion costs per well variants do not report indirect costs consistently NatLabRockies/GEOPHIRES-X#519

Testing & Verification

  1. Unit tests added: Describe new unit tests added, if any.
  2. Unit tests updated: Describe existing unit tests that were updated, if any.
  3. Manual testing: Describe any additional manual testing steps that were performed, such as running the change on a pre-existing data set you own.

Change Checklist

Observe the following checklist items before and after merging, as applicable.

Before merging:

  1. I certify that I am a human and I have manually verified the correctness of the changes in this PR.
  2. Add a CHANGELOG entry if your change includes a new version1 (example). RST Template:
M.m.P: `TITLE <https://github.com/NREL/GEOPHIRES-X/pull/PR-NUMBER>`__ | `release <https://github.com/NREL/GEOPHIRES-X/releases/tag/vM.m.P>`__

After merging:

  1. Push tag(s) to GitHub: git push origin refs/tags/vM.m.P
  2. Post a change announcement to the GEOPHIRES Change Announcements Discussion (example). Markdown template:
 M.m.p: [TITLE](https://github.com/NREL/GEOPHIRES-X/pull/PR-NUMBER) | [release](https://github.com/NREL/GEOPHIRES-X/releases/tag/vM.m.P)

Additional metadata such as TODO items or link to your initial self-review (PR into your own fork created prior to submission)

Footnotes

  1. See versioning notes and version management docs.

@softwareengineerprogrammer

Copy link
Copy Markdown
Owner Author

Rebased: #181

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant