Skip to content

better mixin field checking - #449

Merged
wpbonelli merged 2 commits into
modflowpy:developfrom
wpbonelli:mixin-field-paths
Oct 10, 2026
Merged

wpbonelli merged 2 commits into
modflowpy:developfrom
wpbonelli:mixin-field-paths

Conversation

@wpbonelli

Copy link
Copy Markdown
Member
  • dynamically check generated classes have fields mixin methods require
  • follow record members, inner classes, loop vars, **kwargs dicts with literal keys
  • report gaps per method in flopy4 mf6 status, warn about methods that won't work

Maybe this is overkill but letting the user know there's a problem at sync time is better than waiting til some method is called

wpbonelli and others added 2 commits October 10, 2026 05:09
Scan each mixin's methods for the attribute chains they read from their
class and look them up on the staged classes, in the import check's
subprocess, replacing the hand-kept _requires lists.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@wpbonelli wpbonelli added this to the MVP milestone Oct 10, 2026
@wpbonelli
wpbonelli merged commit 78f84d4 into modflowpy:develop Oct 10, 2026
17 checks passed
@wpbonelli
wpbonelli deleted the mixin-field-paths branch October 10, 2026 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant