-
Notifications
You must be signed in to change notification settings - Fork 56
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
Allow user to set MPI ranks in test suite #15
Comments
Could we build this generally enough that a future test could be cross-validation vs number of ranks? |
That's a great idea. The user could provide a list of ranks and the test case would handle it accordingly. This would actually simplify the interfaces to the tests considerably. Right now, each test case has four methods. For example:
If we allow the user to specify a list of ranks, then we can reduce the test case methods to:
|
Do we also run serial tests w/o MPI? |
We don't, I will add that as a separate issue. |
I added this issue to Nek5000/Nek5000#55 |
Yes - I think this is important, since we advertise that we have this capability From: Ron Rahaman [notifications@github.com] Do we also run serial tests w/o MPI? I added this issue to Nek5000/Nek5000#55https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Nek5000_Nek5000_issues_55&d=CwMCaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=bBKUKHeOUQYUUcfES2GnUlhzzfmhFc7KlYsosI2DIEU&m=IbsD84J6V2u_VNBY0jr_-Z3snBm8c0KVbjI6FdO52jw&s=6H3dxIzpKQ-nFF9jtkg1rrV4rr9yLwjwVAFBJeULWK0&e= — |
Closed in PR #20 |
Currently, the parallel test cases in NekTests.py are always run with 4 MPI ranks. We should allow the user to set the number of MPI ranks.
The text was updated successfully, but these errors were encountered: