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

Skip to content

update ssm package when splitting flow + transport simulations #798

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.

Sign up for GitHub

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

Jump to bottom
Closed
luitjansl opened this issue Feb 1, 2024 · 1 comment
Closed

update ssm package when splitting flow + transport simulations #798

luitjansl opened this issue Feb 1, 2024 · 1 comment
Assignees
@luitjansl
Milestone
Multi-solute tran...

Comments

@luitjansl
Copy link
Contributor

The ssm package is part of a transport model, and it describes what flow boundaries to take into account for transport. For example:

begin options
  save_flows
end options

begin sources
  chd aux species_b
  rch aux species_b
  well aux species_b
end sources

But when splitting into subdomains, not every flow boundary will be present in every partition, and this will cause modflow to fail to run the simulation
@github-project-automation github-project-automation bot moved this to 📯 New in iMOD Suite Feb 1, 2024
@luitjansl luitjansl moved this from 📯 New to 🏗 In Progress in iMOD Suite Feb 1, 2024
@luitjansl luitjansl self-assigned this Feb 1, 2024
@luitjansl luitjansl mentioned this issue Feb 5, 2024
Merged
5 tasks
@luitjansl luitjansl moved this from 🏗 In Progress to 🧐 In Review in iMOD Suite Feb 5, 2024
JoerivanEngelen added a commit that referenced this issue Feb 5, 2024
@luitjansl @JoerivanEngelen
Issue #798 partition ssm packages (#819)
4bc897a 
Fixes #798 

When we split a simulation of both flow and transport models, the ssm
package of the transport simulation is re-generated after splitting,
because not every flow boundary will be present in every partition.


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

---------

Co-authored-by: Joeri van Engelen <[email protected]>
@JoerivanEngelen
Copy link
Contributor

Merged in #819

@github-project-automation github-project-automation bot moved this from 🧐 In Review to ✅ Done in iMOD Suite Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@luitjansl luitjansl

Labels
None yet
Projects
iMOD Suite
Status: ✅ Done
Milestone
Multi-solute transport model support
Development

No branches or pull requests
2 participants
@JoerivanEngelen @luitjansl