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.

Sign up for GitHub

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

Jump to bottom

Change default output type to Float32 #4072

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Change default output type to Float32 #4072

wants to merge 3 commits into from
+14 −8

Conversation

glwagner
Copy link
Member

Supercedes #3214

@navidcy
Copy link
Collaborator

navidcy commented Feb 2, 2025

What's the the out of memory error that started appearing in the CI? E.g.,

https://buildkite.com/clima/oceananigans/builds/20408#0194be91-6568-425d-b131-3efaacd04d66/30-5184

Did anything change recently?

@glwagner
Copy link
Member Author

glwagner commented Feb 3, 2025

https://buildkite.com/clima/oceananigans/builds/20408#0194be91-6568-425d-b131-3efaacd04d66/30-5184

Previously the documentation did not use GPU but we changed that.

@navidcy
Copy link
Collaborator

navidcy commented Feb 3, 2025

Oh but that's been while back... These out of memory errors appeared in the last week or so. We've changed the docs to be using GPU 6 months ago..

@navidcy
Copy link
Collaborator

navidcy commented Feb 3, 2025

Recenty we updated CUDA on tartarus though.. Could that have anything to do with this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simone-silvestri simone-silvestri simone-silvestri approved these changes

@navidcy navidcy navidcy approved these changes

Assignees
No one assigned
Labels
output 💾
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@glwagner @navidcy @simone-silvestri