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

Add a developer test suite that uses an existing conda env #953

Merged
merged 2 commits into from
Apr 8, 2023

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Apr 8, 2023

This suite runs much faster than the full suite because it uses the mpas_dev development environment rather than building conda packages.

This merge also updates test suites to use a new QU240wLI run. This is because I deleted the old one on LCRC by mistake.

This merge also fixes an issue with job dependencies. (Dependent jobs were not correctly being cancelled when the job they depend on failed.)

I deleted the old one on LCRC by mistake.

This merge also fixes an issue where dependencies weren't getting
terminated when the jobs they depend on failed.
This suite runs much faster than the full suite because it uses
the mpas_dev development environment rather than building conda
packages.
@xylar
Copy link
Collaborator Author

xylar commented Apr 8, 2023

Testing

I tested this, along with #954 and #955 in:
https://web.lcrc.anl.gov/public/e3sm/diagnostic_output/ac.xylar/analysis_testing/chrysalis/fix_moc_nans_in_z/

Most tests passed. The no_ncclimo test appears to need a fix to MPAS-Tools related to fill values.

@xylar xylar merged commit e7e5235 into MPAS-Dev:develop Apr 8, 2023
@xylar xylar deleted the add_dev_suite branch April 8, 2023 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant