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

AMG regression test fails with three levels #324

Open
eirikurj opened this issue Oct 11, 2023 · 3 comments
Open

AMG regression test fails with three levels #324

eirikurj opened this issue Oct 11, 2023 · 3 comments

Comments

@eirikurj
Copy link
Contributor

Description

Nightly docker integration tests on the u22-gcc-ompi-stable x86 image is failing on the AMG test (see here). This seems to be a result of a newly merged PR #235.

The test_amg.py:TestAMGReal_2_different_levels_and_iterations.test_convergence AMG regression test is failing to converge.

Steps to reproduce issue

This can be reproduced with the failed image pushed to DockerHub, i.e., mdolab/public:u22-gcc-ompi-stable-amd64-failed (Digest: sha256:76e6eb4be1c823744ec7cfc46fa38e5edc9897b8cf9d54dc08d208cd5db1d2f2). Using the image, download the input test files, and run only the AMG regression tests testflo -s -v -n 1 test_amg.py.

Using the previous commit git checkout 559a1deee07126c521e7a2e18f3968dd0ef7b061, rebuilding and pip installing, then all AMG tests do pass successfully.

Current behavior

Currently, the case does not converge and test fails.

Expected behavior

Case should converge and test pass.

Code versions

Docker image mdolab/public:u22-gcc-ompi-stable-amd64-failed downloaded today (Digest: sha256:76e6eb4be1c823744ec7cfc46fa38e5edc9897b8cf9d54dc08d208cd5db1d2f2)

@sseraj
Copy link
Collaborator

sseraj commented Feb 13, 2024

This was fixed in #325 by changing the test. The root cause is unclear except that using too many AMG levels potentially causes memory-related issues.

@sseraj sseraj closed this as completed Feb 13, 2024
@eirikurj
Copy link
Contributor Author

@sseraj, if there is a known issue, I would prefer to either document it somewhere in the docs or create a separate issue.

@sseraj sseraj changed the title AMG regression test failing AMG regression test fails with three levels Feb 14, 2024
@sseraj
Copy link
Collaborator

sseraj commented Feb 14, 2024

This issue has all the relevant information, so maybe it is best to just reopen this

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

No branches or pull requests

2 participants