Skip to content

Conversation

@benflexcompute
Copy link
Collaborator

@benflexcompute benflexcompute commented Jan 15, 2026

Note

Adds automatic persistence of output units for post-processing user variables.

  • Enhance save_user_variables to derive output_units via batch_get_user_variable_units and persist them in variable_context (converting numbers to expression with dimensionless where applicable)
  • New batch_get_user_variable_units to compute units from default_context values (handles Expression, unyt_array, and numbers)
  • Reuse get_post_processing_variables to scope which variables get unit inference
  • Update tests and reference JSON to include additional variables and assert output_units (e.g., km/ms, m/s, dimensionless)
  • Remove obsolete commented code

Written by Cursor Bugbot for commit cd84b9d. This will update automatically on new commits. Configure here.

@benflexcompute benflexcompute self-assigned this Jan 15, 2026
@benflexcompute benflexcompute merged commit ecfc663 into main Jan 15, 2026
21 checks passed
@benflexcompute benflexcompute deleted the BenY/AugmentOutputUnitsToVariableContext branch January 15, 2026 20:04
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.

2 participants