Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

Merged

Conversation

luitjansl
Copy link
Contributor

Fixes #961

Description

implements importing from a imod5 project file for storage and initial conditions.

Checklist

  • Links to correct issue
  • Update changelog, if changes affect users
  • PR title starts with Issue #nr, e.g. Issue #737
  • Unit tests were added
  • If feature added: Added/extended example

Copy link
Contributor

@JoerivanEngelen JoerivanEngelen left a 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"],
Copy link
Contributor

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.

Copy link
Contributor Author

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

Copy link
Contributor

@JoerivanEngelen JoerivanEngelen left a 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.

luitjansl and others added 2 commits May 8, 2024 15:45
review comment

Co-authored-by: Joeri van Engelen <[email protected]>
review comment

Co-authored-by: Joeri van Engelen <[email protected]>
@luitjansl luitjansl merged commit 65ba5f2 into imod5_converter_feature_branch May 8, 2024
7 checks passed
@luitjansl luitjansl deleted the issue_#961_flow_packages_from_imod5 branch May 8, 2024 15:10
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.

2 participants