Skip to content

Conversation

@wpbonelli
Copy link
Member

@wpbonelli wpbonelli commented Jun 3, 2025

  • move filename field to Component. this required supporting kw_only on xattree classes (passed thru to attrs.define) which lets us put required/optional fields in any order. otherwise attrs dictates required fields must come first therefore an optional filename on Component would mean no required fields on concrete component types.
  • set the default load/write format to ascii on Simulation
  • rename Simulation.path back to workspace as it was originally.. introduce an intermediate component subclass Context which is a component with a workspace
  • make path a derived property on Component and Context

@wpbonelli wpbonelli force-pushed the various branch 4 times, most recently from b1628e0 to eb141a0 Compare June 3, 2025 01:55
@wpbonelli wpbonelli force-pushed the various branch 4 times, most recently from b7705cc to f4260a4 Compare June 3, 2025 02:29
@wpbonelli wpbonelli mentioned this pull request Jun 3, 2025
@wpbonelli wpbonelli merged commit 282e6bd into modflowpy:develop Jun 3, 2025
11 checks passed
@wpbonelli wpbonelli deleted the various branch June 3, 2025 02:55
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.

1 participant