-
Notifications
You must be signed in to change notification settings - Fork 68
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
add parameter to set maximum water saturation. Default is 1.0 #816
Conversation
jenkins build this please |
jenkins build this please |
b86c865
to
75eaee3
Compare
jenkins build this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is acceptable. I would be very nervous about a future PR changing the default, especially for MaxWaterSaturation! However, I acknowledge that this makes it much easier to test various combinations and in turn perhaps find a better solution.
@hnil What do you think?
// special case cells with almost only water | ||
// use both saturations (if the phase is enabled) | ||
// special case for cells with almost only water | ||
//thresholdWaterFilledCell = std::min(thresholdWaterFilledCell, 1.0 - eps); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this?
75eaee3
to
db94249
Compare
jenkins build this please |
Thanks for approving. Will merge. |
This is the second part of
#803
which adds a option to set the maximum water saturation. For some cases using a value slightly above 1.0 (for instance 1.05) improves the convergence significantly. Default is 1.0 so this PR should not change default results.