-
Notifications
You must be signed in to change notification settings - Fork 3
Issue #961 flow packages from imod5 #1023
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
Issue #961 flow packages from imod5 #1023
Conversation
…ckages_from_imod5 # Conflicts: # imod/tests/fixtures/backward_compatibility_fixture.py
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 instructed you wrongly I see now: iMOD5 always lets you define the storage coefficient, instead of specific storage. Sorry for that, could you change this?
imod/mf6/sto.py
Outdated
""" | ||
|
||
data = { | ||
"specific_storage": imod5_data["sto"]["storage_coefficient"], |
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 see I erroneously mentioned to you that iMOD5 always used the specific storage, the exact opposite is true: It is always storage coefficient. This is also mentioned in the iMOD5 manual:
Storage coefficient for each model layer, i.e. the specific storage coefficient multiplied with the thickness of the model layer, for the first unconfined model layer, enter the specific storage coefficient instead, e.g.
0.15.
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.
ok, I'll switch it to storageCoefficient
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.
Approved in advance, have one minor comment.
review comment Co-authored-by: Joeri van Engelen <[email protected]>
review comment Co-authored-by: Joeri van Engelen <[email protected]>
Fixes #961
Description
implements importing from a imod5 project file for storage and initial conditions.
Checklist
Issue #nr
, e.g.Issue #737