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

Switch continental extension to AMG solver. #6003

Merged

Conversation

MFraters
Copy link
Member

@MFraters MFraters commented Aug 6, 2024

Implementation of a simple solution #6002. Although the best fix would be to find the issue why the gmg solver fails, setting it to the amg solver will at least fix the cookbook in the main branch for now.

@gassmoeller
Copy link
Member

I think it is fair to switch back, we should make sure all cookbooks always work. Can you fix the test results?

@MFraters MFraters force-pushed the switch_continental_extension_to_amg branch from c8fc0bb to 90b0c2e Compare August 20, 2024 12:55
@MFraters
Copy link
Member Author

@gassmoeller, I fixed the tests.

Copy link
Member

@gassmoeller gassmoeller left a comment

Choose a reason for hiding this comment

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

👍

@naliboff
Copy link
Contributor

Thanks @MFraters.

@naliboff
Copy link
Contributor

/rebuild

@naliboff naliboff merged commit 5148ea0 into geodynamics:main Aug 20, 2024
7 of 8 checks passed
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.

3 participants