Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Save and Restore Network Node's Lift Gas as Produced Gas Setting #4499

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bska
Copy link
Member

@bska bska commented Feb 26, 2025

This PR ensures that the setting for whether or not the lift gas of a leaf node's corresponding group's subordinate wells should be added to the node's produced gas entering the node in an extended model production network. In short, this commit saves and restores item 4 of NODEPROP for leaf nodes in an extended network.

@bska
Copy link
Member Author

bska commented Feb 26, 2025

I'm creating this PR in draft mode because it depends on, and contains, the earlier PR #4498. I will keep the PR in a draft state until such time as it is ready for review.

@bska bska force-pushed the save-restore-nodeprop-4 branch 4 times, most recently from a61cf30 to 2716272 Compare March 3, 2025 16:41
@bska bska force-pushed the save-restore-nodeprop-4 branch 9 times, most recently from d34f994 to 442e601 Compare March 12, 2025 15:41
This commit ensures that the setting for whether or not the lift gas
of a leaf node's corresponding group's subordinate wells should be
added to the node's produced gas entering the node in an extended
model production network.  In short, this commit saves and restores
item 4 of NODEPROP for leaf nodes in an extended network.
@bska bska force-pushed the save-restore-nodeprop-4 branch from 442e601 to 5a20ab4 Compare March 13, 2025 08:47
@bska
Copy link
Member Author

bska commented Mar 13, 2025

The earlier PR #4498 was merged into the master branch. I'm marking this as "ready for review" and I'm running a build check.

@bska bska marked this pull request as ready for review March 13, 2025 13:29
@bska
Copy link
Member Author

bska commented Mar 13, 2025

jenkins build this please

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