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

(0.90.11) Tests for a distributed Immersed boundary grid + fix some on_architecture bugs #3487

Merged
merged 18 commits into from
Mar 12, 2024

Conversation

simone-silvestri
Copy link
Collaborator

This PR introduces a test to verify the correct execution of a distributed immersed latitude longitude grid

@navidcy navidcy added the testing 🧪 Tests get priority in case of emergency evacuation label Mar 1, 2024
@navidcy
Copy link
Collaborator

navidcy commented Mar 8, 2024

@simone-silvestri, have a look at this error when you find time:
https://buildkite.com/clima/oceananigans-distributed/builds/1464#018e1b66-735b-43bc-9c77-4bb7b4646c2d/192-4782

@navidcy
Copy link
Collaborator

navidcy commented Mar 8, 2024

Bump a patch release?

@navidcy navidcy changed the title Tests for a distributed Immersed boundary grid (0.90.11) Tests for a distributed Immersed boundary grid + fix some on_architecture bugs Mar 11, 2024
@navidcy navidcy self-requested a review March 11, 2024 21:42
@simone-silvestri
Copy link
Collaborator Author

Thanks @navidcy. I ll merge when tests pass

@simone-silvestri simone-silvestri merged commit 3bb62a6 into main Mar 12, 2024
48 checks passed
@simone-silvestri simone-silvestri deleted the ss/test-distributed-immersed-boundaries branch March 12, 2024 02:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing 🧪 Tests get priority in case of emergency evacuation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants