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

Add some extra check #15

Open
JiZur opened this issue Feb 7, 2022 · 0 comments
Open

Add some extra check #15

JiZur opened this issue Feb 7, 2022 · 0 comments

Comments

@JiZur
Copy link

JiZur commented Feb 7, 2022

It would be good to have an initial check that overburden pressure >= fluid pressure and overburden pressure >= reference pressure (i.e effective pore pressure and effective reference pressure are positive!) with understandable error message. That would avoid the current case where Hertz-Mindlin equation crashes due to "invalid value encountered in power function" (due to negative pressure, which is not said) with an error message which does not make the debugging easy.

Since many users use the patchy cement model for dry rock with pressure correction, they neglect to input a reasonable reference pressure (as it is later overwritten by the pressure adjustment), resulting in the intermediate step of dry rock at reference pressure propagating NaN value due to the negative pressure.

@eivindjahren eivindjahren added christmas-review Issues and PRs for Christmas review and removed christmas-review Issues and PRs for Christmas review labels Dec 13, 2024
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

No branches or pull requests

2 participants