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

Fix oscilations #5607

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix oscilations #5607

wants to merge 4 commits into from

Conversation

totto82
Copy link
Member

@totto82 totto82 commented Sep 13, 2024

WIP

Needs a lot more testing....

@totto82
Copy link
Member Author

totto82 commented Sep 16, 2024

jenkins build this please

@totto82
Copy link
Member Author

totto82 commented Sep 20, 2024

This PR with --shut-unsolvable-wells=false restore Norne results

@totto82 totto82 marked this pull request as ready for review September 20, 2024 10:28
@totto82
Copy link
Member Author

totto82 commented Sep 20, 2024

jenkins build this please

@totto82 totto82 requested a review from steink September 20, 2024 10:29
@steink
Copy link
Contributor

steink commented Sep 23, 2024

This PR with --shut-unsolvable-wells=false restore Norne results

Looking into this. Currently a well that has reached it's maximum switching limit is not only prevented from switching control, but also status (e.g., to stop). This leads to convergence failure for e.g., non-feasible bhp. This is a problem with master and not the PR, but probably good to fix in the current PR. Will do some experimentation and propose a fix.

@steink
Copy link
Contributor

steink commented Sep 23, 2024

@totto82 could you make the following small update: aec5170 ? This will prevent forcing a non-operable well to be open in cases the control mode is forced (prevents well convergence failures).

@totto82
Copy link
Member Author

totto82 commented Sep 24, 2024

@totto82 could you make the following small update: aec5170 ? This will prevent forcing a non-operable well to be open in cases the control mode is forced (prevents well convergence failures).

Thanks. I have updated accordingly

@totto82
Copy link
Member Author

totto82 commented Sep 24, 2024

jenkins build this please

@steink
Copy link
Contributor

steink commented Sep 26, 2024

PR fails to complete simulation for MOD4_UDQ_ACTIONX, however I'm not able to reproduce this behaviour locally. Will dig a bit more to try figure out what's going on.

@steink
Copy link
Contributor

steink commented Sep 26, 2024

Found an issue which by coincidense resulted in failed simulation of MOD4_UDQ_ACTIONX for this PR. Will attempt to fix this separately.

@steink
Copy link
Contributor

steink commented Sep 27, 2024

#5634 has been merged, so I think this should fix the trouble for MOD4_UDQ_ACTIONX for this PR. I think the changes here looks good except the comment above. In my opinion maximum-number-of-well-switches can safely get a default value of 10.

@atgeirr
Copy link
Member

atgeirr commented Sep 30, 2024

jenkins build this please

@steink steink added this to the Release 2024.10 milestone Sep 30, 2024
@steink steink removed this from the Release 2024.10 milestone Oct 3, 2024
@steink
Copy link
Contributor

steink commented Oct 3, 2024

See #5641

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.

3 participants