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

Test Wiki Page ambigous about Zoltan requirement for tests #8

Closed
YetAnotherMinion opened this issue Aug 11, 2015 · 11 comments
Closed

Comments

@YetAnotherMinion
Copy link
Contributor

When I run the tests, I encounter below errors

CMakeFiles/ptnParma.dir/ptnParma.cc.o: In function `getPlan':
/home/shivaebola/Documents/GitHub/core/test/ptnParma.cc:68: undefined reference to `apf::makeZoltanGlobalSplitter(apf::Mesh*, int, int, bool)'
collect2: error: ld returned 1 exit status
test/CMakeFiles/ptnParma.dir/build.make:157: recipe for target 'test/ptnParma' failed
make[2]: *** [test/ptnParma] Error 1
CMakeFiles/Makefile2:3006: recipe for target 'test/CMakeFiles/ptnParma.dir/all' failed
make[1]: *** [test/CMakeFiles/ptnParma.dir/all] Error 2
Makefile:147: recipe for target 'all' failed
make: *** [all] Error 2

My steps in directory core

$ git pull
$ cd build
$ source ../config.sh

config.sh is

cmake .. \
  -DCMAKE_C_COMPILER="mpicc" \
  -DCMAKE_CXX_COMPILER="mpicxx" \
  -DCMAKE_C_FLAGS="-O2 -g -Wall" \
  -DCMAKE_CXX_FLAGS="-O2 -g -Wall" \
  -DENABLE_THREADS=ON \
  -DENABLE_ZOLTAN=OFF \
  -DCMAKE_INSTALL_PREFIX="/home/shivaebola/Documents/GitHub/FEP/a4/lib/pumi" \
  -DIS_TESTING=True \
  -DMESHES="/home/shivaebola/Documents/GitHub/core/meshes"

then I build

$ make -j 4

and encounter the above mentioned error based on Zoltan.
It seems to indicate that we need Zoltan to run the test suit, but the wiki does not indicate this requirement. Am I doing it wrong or do I need to link with Zoltan and or ParMetis for tests to run?

Now I am still inside core/build, where I ran make, now running ctest -W, I get only 37% passing

Full results here

@cwsmith
Copy link
Contributor

cwsmith commented Aug 11, 2015

This should be fixed now with 4ff4118 .

@cwsmith cwsmith closed this as completed Aug 11, 2015
@YetAnotherMinion
Copy link
Contributor Author

That fixed the initial make and now ctest finds all of the excecutables. Thank you for fast response!

However I am still failing a worrisome fraction of the build. only 79% passes. Full results here I am using the same settings as above, no changes.

Do I need to change any thing else to pass these other tests?

@cwsmith
Copy link
Contributor

cwsmith commented Aug 11, 2015

You're welcome.

Would you please paste the contents of build/Testing/Temporary/LastTest.log ?

@ibaned
Copy link
Contributor

ibaned commented Aug 11, 2015 via email

@ibaned
Copy link
Contributor

ibaned commented Aug 11, 2015

nevermind, Cameron did go through and disable them properly, and I have confirmed that this works. You probably need to re-configure. So pull, wipe out your build directory, and source the config script again.

@YetAnotherMinion
Copy link
Contributor Author

No such luck, I emptied the build directory, sourced the config, and rebuilt and ran tests. Still seeing same test errors, full build/Testing/Temporary/LastTest.log

@ibaned
Copy link
Contributor

ibaned commented Aug 11, 2015

That log shows that the new problem is your MPI installation doesn't support MPI_THREAD_MULTIPLE. The recommended solution is to change your configuration to -DENABLE_THREADS=OFF.

@YetAnotherMinion
Copy link
Contributor Author

So I need to switch MPI implementations then to get full functionality? I am using OpenMPI 1.8.4 which I had from a previous project.

@cwsmith
Copy link
Contributor

cwsmith commented Aug 11, 2015

You will not lose any functionality with threads disabled.

@ibaned
Copy link
Contributor

ibaned commented Aug 11, 2015

what can be done with threads can also be done with processes, something we are trying to convince many people about. in our code threads are only used for repartitioning from low to high part counts, and we have a process-based implementation of that which is compiled when threads are disabled.

@YetAnotherMinion
Copy link
Contributor Author

All passing now. Thank you for all your help.

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

3 participants