Skip to content

Conversation

@wpbonelli
Copy link
Member

@wpbonelli wpbonelli commented Jun 20, 2025

  • use new xattree builtin dict to array converter, remove custom impl (todo: support tabular data too... xattree should provide a single converter that supports either dict or table input)
  • get rid of star syntax for sparse dict representations of arrays. this was more trouble than it was worth to generalize, and the caller can always use a comprehension to easily generate a still-pretty-sparse dict
  • accommodate get_xatspec refactor (it returns XatSpec now, not the flattened chainmap)
  • miscelleanous other cleanup and notes
  • todo: xattree needs to distinguish array fill value from default. can't always interpret default unambiguously for optional arrays. e.g., when an optional array should be None (i.e. not present) by default, but should have a fill value when populated from a sparse dict or table representation

- use builtin dict to array converter, remove custom impl (todo: support tabular data too)
- get rid of star syntax for sparse dict representations of arrays. this was more trouble than it was worth to generalize
- miscelleanous other cleanup and notes
@wpbonelli wpbonelli merged commit 62aa491 into modflowpy:develop Jun 20, 2025
11 checks passed
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