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

Skip to content

Issue #1457 remove flow module #1458

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
merged 10 commits into from
Feb 26, 2025
Merged

Conversation

JoerivanEngelen
Copy link
Contributor

@JoerivanEngelen JoerivanEngelen commented Feb 25, 2025

Fixes #1457

Removes flow module, it has hardly seen any use. Reduces clutter, scope, potential hurdles to getting familiar with the code base, and potential maintenance.

Description

  • Remove flow module
  • Remove unittests for flow
  • Remove iMODFLOW examples
  • Remove notions that iMOD Python can generate iMODFLOW models from README
  • Update changelog, so that there are no crossrefs to be removed

I also tested generating the docs, so that there are no errors thrown.

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

@JoerivanEngelen JoerivanEngelen added this pull request to the merge queue Feb 26, 2025
Merged via the queue into master with commit e3909c2 Feb 26, 2025
7 checks passed
@JoerivanEngelen JoerivanEngelen deleted the issue_#1457_remove_flow_module branch February 26, 2025 09:42
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.

Remove flow module
2 participants