Update to v2.0.0-alpha.1#944
Conversation
9c93e54 to
0303b90
Compare
0303b90 to
ba7c900
Compare
74bb269 to
cec99ef
Compare
fd971e0 to
7f43434
Compare
|
@matthewhoffman and @trhille, to test this for now, use: This branch is needed until I tag a 3.3.0rc2 for mache. |
|
@xylar , can you walk me through a few more details about the transition to First off, is the mache branch in your previous comment out of date? Mache branch It ran great for awhile and seemed much faster than the old Am I doing this wrong? Is this trying to deploy for the entire project? I don't think you want me interacting with |
|
@matthewhoffman, I'm sorry. I'm developing I needed to release mache 3.3.0 for Polaris yesterday. As a result, the update-to-3.3.0 branch is gone. But I neglected to update this Compass branch until just now. At this point, no You also don't want to deploy spack. That was a mistake in my command above. |
d3cc399 to
21d2713
Compare
|
Thanks, @xylar . I made a little more progress with the command you suggested. I had to make this change: but then I still ran into an issue of it trying to touch the deployed spack env in the e3sm project space: |
I think that's in 21d2713. Did you not have that commit or did I miss something? |
Yep, that's something I need to fix. Sorry about that! |
|
@matthewhoffman, the second issue should be fixed. |
|
@xylar , thanks for addressing the second issue. The first must have been because I had failed to update my local branch this morning. After updating to 160d75d , |
The Compass version is in there: It's just called something different than before. We can copy that into another environment variable if you need it. |
|
Oh, wait, it already is: |
|
Are you not seeing that in you load script? |
|
I just mean the name of the load script used to have the version in the filename, but I'm not seeing that. It's not a big deal, I was just wondering if that was intentional. As for progress, when I compile MALI I am seeing the same PIO lib errors that you do in the issue you opened. I'm working on debugging them with help from ChatGPT and so far the obvious things are not working, but I'll keep at it while I have time. |
|
I see. No, the load script won't include the compass version anymore. I didn't find that to be particularly useful. |
It is too old. Don't exclude it on Perlmutter, where it is new enough and can't be built with Spack.
These clash with MPAS framework's equivalent files.
This keeps ESMF from stepping on SCORPIO's toes by installing its own ParallelIO.
This is no longer needed now that ESMF is in the software environment.
069483c to
d121b6b
Compare
|
@xylar and @mperego , I think we should go ahead and remove the exodus output from the tests. We had added it at some point because when runs fail it is sometimes useful to be able to look at the velocity solution on the exo mesh to see what's going on, and it's convenient to not have to rerun the tests. But I can't remember the last time I've actually had to look at them, so it's not a big inconvenience to disable them again, and that seems a much better use of time than trying to debug these libraries. @xylar , are you ok if I push a commit to your branch that makes the changes to disable the exo output? That way we can test everything in this branch. |
|
Also, is there anything I should be aware of in your push since the last discussion messages? Should I rebuild my env locally before testing again? |
Yes, go for it. |
Yes, you need to rebuild. I switch to a much newer version of mache. I think I rebuild the spack environments but it's hard to keep track of everything right now :-( |
|
@xylar , I'm getting a lot of failures, and it looks like problems are occurring in mpas_tools in It looks like every test that is creating a mesh is failing on that step. Here is the first example (landice_dome_2000m_sia_restart_test): |
These versions fix an issue with copying datasets in the latest xarray.
|
@matthewhoffman, sorry about that. These are fixed in the latest mpas_tools and geometric_features. I have switched to those versions. Another case of the redundant maintenance burden between Compass and Polaris meaning that things get lost, I'm afraid. |
The new `mpas_tools` has dropped python 3.10
|
No problem, @xylar! Thanks for updating the branch so quickly. I'm no longer seeing that error during mesh generation. I'll give a broader update once my tests complete and I've had a chance to apply the update disabling the exodus output. |
Due to issues with i/o library linking in this branch, Albany is dying with FPE when MALI is compiled in debug mode. The easiest solutoin is to disable Exodus output being created by default in MALI's compass tests. If one needs to use it, they will need to compile MALI in non-debug mode (or at least without the -ffpe-trap compiler flag.
|
After a few false starts, I managed to successfully test this branch with the full_integration suite on pm-cpu, using a baseline of compass= All tests without Albany pass, including baseline comparison. All tests with with Albany pass execution and validation but fail baseline comparison. This is expected with the updated Albany and Trilinos versions. I checked that the baseline comparison diffs are expectedly small: <1e-3 m for full_integration resultsWith this set of tests, this branch is approved for current functionality! Before approving the PR, I still need to test the new debris friction functionality in MALI, which will require a throwaway merge of #938 and associated MALI throwaway merge. |
Testing with debris-friction updatesThis comment describes throwaway testing for the in-waiting debris-friction feature in MALI and in-waiting debris-friction tests in compass. Testing is performed by merging each of those branches into the respective branches for MALI and compass used in the previous comment. These merges will occur after this v2.0.0-alpha.1 env is merged, but I need to ensure the Albany updates in this PR work correctly with those features, because that was the original reason for updating Albany in compass that led to this PR. Results of new debris friction tests: Based on this, it appears everything is working as intended. |
matthewhoffman
left a comment
There was a problem hiding this comment.
Approving based on test results. I've only cursorily skimmed the changes.
This pull request updates to
mache.deploy, which uses the./deploy.pyscript instead of./conda/configure-compass-env.py.It switches to using
pixiin the background for creating environments with conda packages.Updates:
mache.deploy,mache.jigsawandmache.parallelas well as module updates on many machines and several bug fixescompass-2026-03-21compass-2026-02-06Testing
Only testing MALI, as MPAS-Ocean is no longer being tested regularly on Compass.
MALI with
full_integration:Deployed
MALI with
full_integration: