-
Notifications
You must be signed in to change notification settings - Fork 110
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
Add more tests to Travis #40
Comments
I agree |
Also we need to audit Test_simd for more functionality, as you hit some test escapes. |
I already added the tests for the real and imag (which for consistency with the vectorization do NOT return a real object but a complex - a real can be obtained with the toReal function). |
we also need an mpi build and two mpi task sanity check. |
Two things:
|
MPI does not look to go well with Travis. The Linux version uses an archaic MPI which misses some types and the OS X one crashes just calling |
the build log on MacOSX looks like it built ok but was missing the --mpi 1.2.1.1 flag on the execution to indicate the node grid? It might be ok ? |
Agree the MPI_UINT32_T is weird on Travis/Linux. |
Arrrg, of course... I will correct that. The version of MPI Travis is using is http://packages.ubuntu.com/precise/libopenmpi-dev, I am not sure how old it is (but probably a lot). |
Ok I made that work. There was of course the missing |
MPI tests are working and merged into |
Correct me if I'm wrong, but I think this issue should still be opened as we still need a |
-- Tests/forces Fermion and Gauge Force tests across range of actions |
Test_cshift_red_black |
I've discovered: AM_EXTRA_RECURSIVE_TARGETS and will use targets "check, bench, and tests" as recursive in the way that "all" is already The autogenerated Make.inc, created by scripts/filelist tests-local: for subdirs under Grid/tests. We need in the local Makefile.am only the following: (e.g. tests/Makefile.am + subdir Makefile.am's) (e.g. benchmarks/Makefile.am) There is no need for makefiles to contain a shell script over subdirs. It is possible to introduce (easily) now a "make extras" target that recurses appropriately. Adding a new test to make check involves just adding to a check-local: To the local Makefile.am. Only short running tests that check their answer and throw error by calling assert( answer == good ) ; This results in a make fail as below. I use assert and not exit(-1) etc because this throws Note the build is autorecursive. c010200:build peterboyle$ make check |
I suggest to add few more tests to the Travis CI:
The text was updated successfully, but these errors were encountered: