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

Always use ClimaCommon modules #3317

Closed
wants to merge 1 commit into from

Conversation

charleskawczynski
Copy link
Member

Perhaps this is why we've been getting inconsistent timings on buildkite (and why some of our target gpu runs have been broken).

@Sbozzolo
Copy link
Member

climacommon is mainly a module on the Caltech cluster. On clima, it is just an alias for the common module, provided for convneinence and muscle-memory. The common module does not load nsight, so this would fail. When you load nsight, this is be perfectly equivalent to explicitely loading the modules.

The reason I didn't change them to climacommon is that it would require ensuring that every new release of climacommon also exists on clima (at least in name, even if they are aliases). Given that clima does modules a little differently, and is a more stable platform and doesn't see/need much updates, I didn't think it was necessary to add extra infrastructure/points of failure there.

I would like to move clima to more consistent modules with the cluster, but that would require some work.

@charleskawczynski
Copy link
Member Author

Ok, makes sense. Synchronizing at the moment isn't too bad, but I hope will can somehow make synchronizing even smoother.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants