-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Align Action methods with coupler methods #3096
Comments
tclune
added a commit
that referenced
this issue
Oct 24, 2024
This commit provides low level capabilities for computing the difference in metadata (units, typekind, geom, ...) between 2 fields or field bundles as well as applying diffs to a targeted field/bundle. The intent is for couplers to use this to propagate time-varying attributes between import and export. Changes will generally flow in each direction. Intermediate progress. May have to start over splitting into finer tasks. Update/reallocate on fields moved - New class FieldDelta and modified tests now in Test_FieldDelta.pf. - FieldBundleDelta only partially completed. Intermediate progress on FieldBudleDelta tests Tests pass. Still need more tests to check treatment of ungridded dims. FieldDelta and FieldBundleDelta pass tests. Have commented out logic for controlling time varying - saving for later feature.
tclune
added a commit
that referenced
this issue
Oct 24, 2024
tclune
added a commit
that referenced
this issue
Oct 24, 2024
…lidate_phase' into feature/tclune/#3096-action-invalidate_phase
tclune
added a commit
that referenced
this issue
Oct 27, 2024
tclune
added a commit
that referenced
this issue
Oct 28, 2024
tclune
added a commit
that referenced
this issue
Oct 28, 2024
…validate_phase Fixes #3096 - propagate time dependent changes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The task went a bit sideways so the original title and description are wrong. The work done is associated with the original title, but the specific actions were handled in a separate PR.
The text was updated successfully, but these errors were encountered: