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

1deg_jra55do_ryf: update OM3 executable to latest version #53

Merged
merged 3 commits into from
Apr 7, 2024

Conversation

micaeljtoliveira
Copy link
Contributor

Changes needed to use latest OM3 executable.

Contributes to COSIMA/access-om3#118

… using a spack built executable, all dependencies are handled through rpath and we can leave payu to figure out whatever is missing.
@micaeljtoliveira micaeljtoliveira added the 1deg_jra55do_ryf 1deg_jra55do_ryf configuration label Apr 5, 2024
Copy link
Collaborator

@dougiesquire dougiesquire left a comment

Choose a reason for hiding this comment

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

LGMT and ran successfully

@micaeljtoliveira
Copy link
Contributor Author

Marking this as a draft, as I'll have to update the executable again, as I need to reinstall OM3 after COSIMA/access-om3#131

@micaeljtoliveira micaeljtoliveira marked this pull request as draft April 5, 2024 05:29
@micaeljtoliveira micaeljtoliveira marked this pull request as ready for review April 7, 2024 22:52
@micaeljtoliveira
Copy link
Contributor Author

Merging this as is. I'm going to change the way how we specify the executable path so that we don't have to update all configurations whenever we update the development executable and I prefer to do it in a separate PR.

@micaeljtoliveira micaeljtoliveira merged commit 1bad3ae into 1deg_jra55do_ryf Apr 7, 2024
@micaeljtoliveira micaeljtoliveira deleted the 1deg_jra55do_ryf_iss118 branch April 7, 2024 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1deg_jra55do_ryf 1deg_jra55do_ryf configuration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants