-
Notifications
You must be signed in to change notification settings - Fork 80
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
Reenabling unit tests for Pangea2 target #1245
Comments
Before it's deleted, here is the log. Only
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Alright, I mixed up things with multiple containers. The current one has the proper |
Found this open-mpi/ompi#4948 about our
I do have a lot of this kind of messages (on export OMPI_MCA_btl_vader_single_copy_mechanism=none and I do not have this message again. Good. I shall try to understand what's going on and propose a fix for this. Then in my test I can now see (which was already there but hidden by so much noise)
I know it's for
|
I think we had already done this (see discussion here): https://github.com/GEOSX/GEOSX/blob/43fc91e51e5d9be51457d4dee49e13c613f025db/.travis.yml#L47 |
This is definitely worth investigating more. |
Oh yes, i did not copy it 🤦 Error is from https://github.com/trilinos/Trilinos/blob/15dfd41034f5d77060c201a22dc9ecca9cff2293/packages/ml/src/Coarsen/ml_agg_uncoupled.c#L368 , in function |
Another element I should have mentioned (maybe supporting you size error concern). At some point I was able to have the Pangea2 target passing on Travis by adding for debugging purposes GEOSX_LOG_RANK_VAR( partitioning[0] );
GEOSX_LOG_RANK_VAR( partitioning[1] ); at line 162 below |
Hi @castelletto1 LinearSolverParameters params_Direct()
{
LinearSolverParameters parameters;
parameters.solverType = geosx::LinearSolverParameters::SolverType::direct;
parameters.direct.parallel = 0;
return parameters;
} |
Note that option For testing physics solver capabilities (see integrated tests), umpfack is preferable since it can be used with default options. This is not the case for SuperLU_Dist. |
Thx @castelletto1 I gave a shot at So there should something wrong emphasized by Do you have any idea about that? |
Originally that unit test had |
Live debugging from |
Unit test
testLAOperations
fails on Pangea2 target. See discussion #1239 (comment).The text was updated successfully, but these errors were encountered: