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

Updating to SCORPIO v1.6.7 #6881

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

jayeshkrishna
Copy link
Contributor

SCORPIO v1.6.7 includes the following fixes,

  • Fix for E3SM restart issues with ADIOS
  • CMake fixes (reduces warnings, workaround for CMake 3.22 bug)
  • Support for ADIOSC in legacy Fortran interface

[BFB]

SCORPIO v1.6.7 includes the following fixes,

* Fix for E3SM restart issues with ADIOS
* CMake fixes (reduces warnings, workaround for CMake 3.22 bug)
* Support for ADIOSC in legacy Fortran interface
@jayeshkrishna jayeshkrishna added BFB PR leaves answers BFB SCORPIO The E3SM I/O library (derived from PIO) labels Jan 9, 2025
@jayeshkrishna jayeshkrishna requested a review from dqwu January 9, 2025 20:56
@jayeshkrishna jayeshkrishna self-assigned this Jan 9, 2025
Copy link
Contributor

@bartgol bartgol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need an snl-testing member approval to run EAMxx tests

@rljacob
Copy link
Member

rljacob commented Jan 24, 2025

@jayeshkrishna the CI testing shows some fails for eamxx.

@jayeshkrishna
Copy link
Contributor Author

jayeshkrishna commented Jan 24, 2025

@bartgol : Do you think the eamxx fails above are related to SCORPIO? I see a lot of EKAT assertion fails like,

/home/runner/_work/E3SM/E3SM/externals/ekat/src/ekat/kokkos/ekat_subview_utils.hpp:33: Kokkos::View<Kokkos::View<ST *, Kokkos::LayoutRight, Props...>::traits::scalar_array_type, Kokkos::View<ST *, Kokkos::LayoutRight, Props...>::traits::array_layout, Kokkos::View<ST *, Kokkos::LayoutRight, Props...>::traits::device_type, Kokkos::MemoryTraits<<expression>>> ekat::subview(const Kokkos::View<ST **, Kokkos::LayoutRight, Props...> &, int) [with ST = const float; Props = Kokkos::Device<Kokkos::Cuda, Kokkos::CudaSpace>; Props = Kokkos::MemoryTraits<0U>]: block: [124,0,0], thread: [0,0,0] Assertion `i0>=0 && i0 < v.extent_int(0)` failed.

@mahf708
Copy link
Contributor

mahf708 commented Jan 26, 2025

I don't think the errors in the CI are related to SCORPIO; they seem to be related to other known errors (mam wetscav, microphys, etc.)

@bartgol
Copy link
Contributor

bartgol commented Jan 27, 2025

Yeah, I think the errors came from an old bug in mam, which got recently resolved. I will trigger a new set of testing to be sure.

@bartgol bartgol marked this pull request as draft January 27, 2025 16:34
@bartgol bartgol marked this pull request as ready for review January 27, 2025 16:34
@jayeshkrishna
Copy link
Contributor Author

jayeshkrishna commented Jan 29, 2025

@bartgol : The test cosp_standalone_baseline_cmp is failing in the EAMXX test checks below, the fail is a DIFF in comparison with baselines. Is this related to this PR (or is a separate known issue)?

@bartgol
Copy link
Contributor

bartgol commented Jan 29, 2025

@jayeshkrishna there was a pr that fixed cosp that went in last week. I think I forgot to bless master until Monday. I'm rerunning the failed tests, in case the CI for this PR ran before the blessing on Monday. Will know soon enough.

Edit: looks like they all pass now. Good to go on the eamxx side.

jayeshkrishna added a commit that referenced this pull request Jan 30, 2025
…6881)

SCORPIO v1.6.7 includes the following fixes,

* Fix for E3SM restart issues with ADIOS
* CMake fixes (reduces warnings, workaround for CMake 3.22 bug)
* Support for ADIOSC in legacy Fortran interface

[BFB]

* jayeshkrishna/externals/scorpio-v1.6.7:
  Updating to SCORPIO v1.6.7
@jayeshkrishna jayeshkrishna merged commit c311a52 into master Jan 31, 2025
21 of 28 checks passed
@jayeshkrishna jayeshkrishna deleted the jayeshkrishna/externals/scorpio-v1.6.7 branch January 31, 2025 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB SCORPIO The E3SM I/O library (derived from PIO)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants