-
Notifications
You must be signed in to change notification settings - Fork 21
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
Release: 24.12 #771
Release: 24.12 #771
Conversation
Hi, just FYI: CF release fails
|
We still need a |
Hm. We intentionally had to skip updating the pyAMReX & AMReX dependencies for this release, because of some issues with Windows + GPU memory management on the AMReX side. However, all the ImpactX tests pass in CI using the existing (previous) dependencies. I didn't realize updating these dependencies was mandatory. (Maybe these feedstock tests should be run before the release is published?) Is there some way to keep the old dependencies on the |
Yes, we mostly keep the versions in sync to avoid subtle binary (ABI) incompatibilities that we have no CI and developer time for.
Can you link those just for reference? Curious. Note that we do not yet build packages for GPU on CF releases.
Yes, one can temporarily patch ImpactX or pyAMReX on CF, i.e., this line is intentionally set to COMPATIBILITY SameMinorVersion and currently triggers the CF build release error in ImpactX 24.12 for CF:
I recommend just patching the ImpactX line in https://github.com/ECP-WarpX/impactx/blob/24.12/cmake/dependencies/pyAMReX.cmake#L62 |
That's a bit of a chicken-egg problem. We could do pre-releases to run though package managers (Spack, CF, etc.) and we do have general tests that build with the same compilers, but the exact release has to happen at some point. All in all no big deal, we just need the release scripts I mentioned above to automate for us so we have an easier time and don´t need to remember such details :) |
Prepare the December release of ImpactX.
Delay updating the AMReX and pyAMReX dependencies for now, pending:
#760
#769