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

CI is not using icepack #968

Open
travissluka opened this issue Oct 23, 2023 · 2 comments
Open

CI is not using icepack #968

travissluka opened this issue Oct 23, 2023 · 2 comments
Labels
bug Something is not working SOCA Sea-ice, Ocean, and Coupled Assimilation

Comments

@travissluka
Copy link
Collaborator

icepack is optional, but is enabled in the TravisCI tests ( or at least it was supposed to have been enabled, see #967).

It seems icepack is not being built with the new CI. Turn it on.

@travissluka travissluka added bug Something is not working SOCA Sea-ice, Ocean, and Coupled Assimilation labels Oct 23, 2023
@shlyaeva
Copy link
Contributor

shlyaeva commented Dec 2, 2024

This would be nice. Do you think we'd need an update to jedi-bundle and an option in CI to build it, or build it as part of spack-stack?

@travissluka
Copy link
Collaborator Author

This would be nice. Do you think we'd need an update to jedi-bundle and an option in CI to build it, or build it as part of spack-stack?

Yes, the first case: the icepack repo (the jcsda fork of it) should be added to the jedi-bundle (

ecbuild_bundle( PROJECT icepack GIT "https://github.com/JCSDA-internal/Icepack.git" UPDATE BRANCH feature/ecbuild-new )
) and turned on in the CI. I just haven't had time to look into it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working SOCA Sea-ice, Ocean, and Coupled Assimilation
Projects
None yet
Development

No branches or pull requests

2 participants