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

Find active cells with idomain > 0 instead of idomain == 1 #1376

Open
JoerivanEngelen opened this issue Jan 10, 2025 · 0 comments · May be fixed by #1377
Open

Find active cells with idomain > 0 instead of idomain == 1 #1376

JoerivanEngelen opened this issue Jan 10, 2025 · 0 comments · May be fixed by #1377
Assignees
Labels
bug Something isn't working

Comments

@JoerivanEngelen
Copy link
Contributor

JoerivanEngelen commented Jan 10, 2025

The LHM apparently has > 1000 cells where IDOMAIN == 2. I noticed that wells located in these cells get masked away. For both iMOD5 as well as MODFLOW 6, IDOMAIN > 0 means active cell. iMOD Python, however, enforces/assumes IDOMAIN == 1, but we don't actually validate this.

I think it is more user friendly to support IDOMAIN > 0. A quick search showed that we only have a few files where active is computed as idomain == 1, so it appears to be low hanging fruit.

@JoerivanEngelen JoerivanEngelen added the bug Something isn't working label Jan 10, 2025
@github-project-automation github-project-automation bot moved this to 📯 New in iMOD Suite Jan 10, 2025
@JoerivanEngelen JoerivanEngelen self-assigned this Jan 10, 2025
@JoerivanEngelen JoerivanEngelen moved this from 📯 New to 🏗 In Progress in iMOD Suite Jan 10, 2025
@JoerivanEngelen JoerivanEngelen changed the title [Bug] - Find active cells with idomain > 0 instead of idomain == 1 Find active cells with idomain > 0 instead of idomain == 1 Jan 10, 2025
@JoerivanEngelen JoerivanEngelen linked a pull request Jan 10, 2025 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 🏗 In Progress
Development

Successfully merging a pull request may close this issue.

1 participant