Skip to content

feat: tolerate unknown options#2809

Draft
wpbonelli wants to merge 2 commits into
MODFLOW-ORG:developfrom
wpbonelli:tolerate-unknown-options
Draft

feat: tolerate unknown options#2809
wpbonelli wants to merge 2 commits into
MODFLOW-ORG:developfrom
wpbonelli:tolerate-unknown-options

Conversation

@wpbonelli
Copy link
Copy Markdown
Member

@wpbonelli wpbonelli commented May 21, 2026

Add a keyword option TOLERATE_UNKNOWN to the simulation, defaulting false. If present, unrecognized options will not stop the simulation with an error, rather the simulation will continue with a warning.

Motivated by modflowpy/flopy#2710. Alternative and/or complimentary solution to modflowpy/flopy#2712. Still TBD what to do here.


Checklist of items for pull request

  • Replaced section above with description of pull request
  • Added new test or modified an existing test
  • Ran ruff on new and modified python scripts in .doc, autotests, doc, distribution, pymake, and utils subdirectories.
  • Formatted new and modified Fortran source files with fprettify
  • Added doxygen comments to new and modified procedures
  • Updated definition files
  • Updated develop.toml with a plain-language description of the bug fix, change, feature; required for changes that may affect users
  • Updated input and output guide
  • Removed checklist items not relevant to this pull request

@wpbonelli wpbonelli added this to the 6.8.0 milestone May 21, 2026
@wpbonelli wpbonelli added enhancement option MF6 IO related labels May 21, 2026
@wpbonelli wpbonelli requested a review from mjreno May 21, 2026 13:14
@wpbonelli wpbonelli added the question Needs someone's attention label May 25, 2026
this%mf6_input%subcomponent_type, &
this%mf6_input%block_dfns(iblk)%blockname, &
tag, this%filename)
if (itolerate_unknown == 1) then
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't specific to the OPTIONS block you would also need to check the block here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement option MF6 IO related question Needs someone's attention

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants