-
Notifications
You must be signed in to change notification settings - Fork 1
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
1deg_jra55do_iaf: update OM3 executable to latest version #22
Conversation
Unfortunately this currently does not run. It crashes with the following error:
@ezhilsabareesh8 Do you know how to fix this? |
Yeah, I think this configuration is currently broken for the current switches - see #17 |
Thanks. Shall we then simply merge this as is, without testing? Or leave it open or now? |
Hi @micaeljtoliveira , the executables needs to be updated in the bin directory to match the |
@ezhilsabareesh8 I'm using spack to build and install the executables, not the $ ls /g/data/ik11/spack/0.21.2/opt/linux-rocky8-cascadelake/intel-2021.10.0/access-om3-main-6pue372/bin/
access-om3-CICE6-WW3 access-om3-MOM6-CICE6 access-om3-MOM6-CICE6-WW3 ww3_grid ww3_ounf ww3_ounp ww3_outf ww3_strt Edit: the latest version of the |
@micaeljtoliveira This branch 1deg_jra55do_iaf_iss118 is compatible only with the WW3_switches access-om3 build, the changes are not pushed to |
Ah, okay. Then I think it's best to leave this open until you're done with updating the switches. |
I have created a pull request in |
@ezhilsabareesh8 I've now tried to run this branch with the OM3 version that includes the changes from COSIMA/access-om3#131, but I still get the same error. As we have also updated the WW3 version in OM3, don't we need to regenerate the |
… using a spack built executable, all dependencies are handled through rpath and we can leave payu to figure out whatever is missing.
976a129
to
cb41d3d
Compare
Hi @micaeljtoliveira, yes |
@ezhilsabareesh8 Thanks! Seems to work correctly now. |
However, I found something weird from the
|
@ezhilsabareesh8 I suggest we merge this nevertheless, and fix any remaining issue in a different PR. |
Sounds good to me, I am looking into this issues, will open a separate PR |
This PR can be closed, I opened a separate issue #143 to address the changes with updated WW3 component. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. The updated OM3 executable is working and the compatible mod_def.ww3
and restart.ww3
are added to ik11/inputs/
directory.
Changes needed to use latest OM3 executable.
Contributes to COSIMA/access-om3#118