-
Notifications
You must be signed in to change notification settings - Fork 577
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
Zoltan2: Several tests fail with 64 bit builds of of Scotch and ParMETIS #476
Comments
The same as for similar failing Zoltan tests in #475, I think the best short-term solution is option-1 "Disable only the currently failing tests for just the SEMS Dev Env build". NOTE: These tests will only be disabled for the SEMS Dev Env build and no other builds. I will provide reproducible instructions once I push the commit for the SEMSDevEnv.cmake file that disables these tests. @trilinos/framework, FYI: we need all clean Zoltan2 tests in pre-push CI testing. |
Note that the situation for Zoltan2 is different from Zoltan w.r.t. to enabling 64-bit versions of Scotch and ParMETIS. In the case of Zoltan shown in #475, enabling Scotch and ParMETIS only enables several new tests (about 1/2 of which actually pass). So there is no downside in enabling 64-bit versions of Scotch and ParMETIS and then (temporarily) disabling the ones the don't pass because all of the tests without Scotch and ParMETIS enabled still pass. But for Zoltan2, several tests that actually do pass with Scotch and ParMETIS disabled fail when you enable the 64 bit versions of these as shown here: http://testing.sandia.gov/cdash/viewTest.php?onlypassed&buildid=2487199 (e.g. see passing tests What is even more interesting is that there are several Zoltan2 tests shown in the above CDash page that have "ParMETIS" in the name and are shown as passing in the query: but ParMETIS is clearly not enable if you look at the configure for this build here: http://testing.sandia.gov/cdash/viewConfigure.php?buildid=2487199 which shows: Final set of non-enabled TPLs: ... Scotch ... ParMETIS ... 93 How can ParMETIS tests pass if ParMETIS is not even enabled? It appears from the test output that these tests are actually failing but the test is reporting as passing because it is printing "PASS". Take for example the test http://testing.sandia.gov/cdash/testDetails.php?test=34046494&build=2487199 which shows the output:
One might argue that it is better to not enable a test than to have it pass in a trivial way. Anyway, given all of this, I think it is best to just not yet enable Soctch and ParMETIS testing for Zoltan2 until the tests can be fixed for 64 bit versions of these libraries. Again, as I noted in #475, there are no current automated builds of Trilinos that appear to be enabling Scotch or ParMETIS so Zoltan2 will not be loosing any testing. |
Currently if you eanble 64-bit Scotch and ParMETIS with Zoltan2 several tests fail. This is a known issue and these tests will be fixed soon. After that, this commit can be reverted. Also, you need to disable ParMETIS in ShyLU as well.
This enables ParMETIS with passing tests for Amesos, Amesos2, ML, and SEACAS (see Trilinos #158). Currently these 64-bit TPLs are not enabled for Zoltan and Zoltan2 in this build because the Zoltan and Zoltan2 test suites don't currently work with 64-bit libraries (see Trilinos #475 and #476). Also, ShyLU support for ParMETIS is also not enabled because it needs ParMETIS suppot from Zoltan2 which is not enabled. Once the Zoltan and Zoltan2 test suites using 64 bit TPLs are fixed, then these TPLs can be enabled. Note that the ShyLU test for ParMETIS passes for the 64 bit ParMETIS so nothing in ShyLU needs to be fixed. Note that SEMS only provides MPI builds of these TPLs so they are disabled for serial builds. Build/Test Cases Summary Enabled Packages: Disabled Packages: PyTrilinos,Pliris,Claps,TriKota Enabled all Packages 0) MPI_DEBUG => Test case MPI_DEBUG was not run! => Does not affect push readiness! (-1.00 min) 1) SERIAL_RELEASE => Test case SERIAL_RELEASE was not run! => Does not affect push readiness! (-1.00 min) 2) MPI_RELEASE_DEBUG_ST => passed: passed=2346,notpassed=0 (341.29 min) 3) SERIAL_RELEASE_ST => passed: passed=2163,notpassed=0 (172.52 min) Other local commits for this build/test group: 92a1d8d, f2b3c92
Zoltan2 Developers, To reproduce the failing Zoltan2 tests building against the 64-bit ParMETIS and Scotch TPLs in the SEMS Dev Env in order to fix them, one just needs to be on a machine that provides the SEMS Dev Env and then do something like the following:
If that does not work to reproduce the failing tests shown above, please let me know. Once all of the Zoltan tests are passing, then the commit 92a1d8d just needs to be reverted using:
Then, if desired, a Zoltan2 developer could test and push these changes on a machine with the SEMS Dev Env with:
Thanks, -Ross |
Zoltan2 is working fine with ParMETIS. The tests that are failing above are Scotch tests. I verified that Zoltan2 works with proper installations of ParMETIS and Scotch (32-bit or 64-bit). Thus, I will close this issue and re-enable the tests. |
Scotch should be compiled without pthreads support for all our purposes. |
…476) Now that Zoltan2 should be compatible with 64-bit ParMETIS, we should be able to allow the enable of ParMETIS with Zoltan2 and ShyLU. Note that the 32-bit Scotch is still disabled globally (see Trilinos #158 and $476 for details). Build/Test Cases Summary Enabled Packages: Zoltan2, ShyLUCore Disabled Packages: PyTrilinos,Pliris,Claps,TriKota Enabled all Forward Packages 0) MPI_DEBUG => Test case MPI_DEBUG was not run! => Does not affect push readiness! (-1.00 min) 1) SERIAL_RELEASE => Test case SERIAL_RELEASE was not run! => Does not affect push readiness! (-1.00 min) 2) MPI_RELEASE_DEBUG_SHARED_ST => passed: passed=696,notpassed=0 (85.66 min) 3) SERIAL_RELEASE_SHARED_ST => passed: passed=548,notpassed=0 (11.34 min) 4) MPI_RELEASE_DEBUG_STATIC_ST => passed: passed=696,notpassed=0 (49.19 min) 5) SERIAL_RELEASE_STATIC_ST => passed: passed=548,notpassed=0 (10.31 min)
NOTE: With the commit d960992, now ParMETIS is enabled for Zoltan2 and ShyLU for the SEMS Dev Env builds. This enables and runs the tests:
|
Next Action Status:
64-bit Scotch and ParMETIS not enabled for Zoltan2 yet. Next: Zoltan2 team to fix failing tests then enable ...
CC: @trilinos/zoltan2
Description:
As @kddevin predicted in this #158 comment, several of the Scotch and ParMETIS tests fail when using a 64 bit build of Scott and ParMETIS. These are the only TPLs that are available with the SEMS Dev Env (see a lengthy discussion in #158).
In particular, the following Zoltan2 tests failed with the 64 bit builds of Scotch and ParMETIS:
However, what is interesting is that several Zoltan2 "scotch" and "parmetis" tests also passed:
There are many possible options to address these failing tests that I can think of:
<test_name>_DISABLE=TRUE
in the SEMSDevEnv.cmake file.Zoltan2_ENABLE_Scotch=OFF
andZoltan2_ENABLE_ParMETIS=OFF
in the SEMSDevEnv.cmake file.I will provide detailed reproducibility instructions in a later comment.
Definition of Done:
Tasks:
???
The text was updated successfully, but these errors were encountered: