Skip to content
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

Update to 1.7.0 alpha.2 #881

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Jan 9, 2025

This brings in:

  • mpas_tools v1.0.0 -- modernizes command-line tools (including renaming them)

As a result of the mpas_tools update, many command-line tools used by landice have been renamed. The relevant documentation has also been updated.

Checklist

  • User's Guide has been updated
  • Developer's Guide has been updated
  • Documentation has been built locally and changes look as expected
  • Document (in a comment titled Testing in this PR) any testing that was used to verify the changes

@xylar xylar added documentation Improvements or additions to documentation land ice clean-up dependencies and deployment Changes relate to creating conda and Spack environments, and creating a load script labels Jan 9, 2025
@xylar xylar self-assigned this Jan 9, 2025
@xylar
Copy link
Collaborator Author

xylar commented Jan 9, 2025

This needs:
MPAS-Dev/MPAS-Tools#596
MPAS-Dev/MPAS-Tools#597
and a new release of MPAS-Tools (v1.0.0) before it will be useful.

@xylar xylar force-pushed the update-to-1.7.0-alpha.2 branch from 62fd06a to fe92eb2 Compare January 11, 2025 15:03
@xylar xylar force-pushed the update-to-1.7.0-alpha.2 branch from c27fa8a to c30a74c Compare January 29, 2025 10:38
@xylar
Copy link
Collaborator Author

xylar commented Jan 29, 2025

Testing

I ran both the pr and full_integration suites on Chrysalis with this new version of mpas_tools and the associated changes in this branch. I didn't check that thing were BFB however and I need to do that.

@xylar
Copy link
Collaborator Author

xylar commented Jan 29, 2025

@trhille and @matthewhoffman, I will need your help to make sure I didn't break anything for you 2. Can you run tests that go beyond full_integration to make sure other affected tests aren't broken? Then, can you review this and MPAS-Dev/MPAS-Tools#596 together?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clean-up dependencies and deployment Changes relate to creating conda and Spack environments, and creating a load script documentation Improvements or additions to documentation land ice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant