[SyneRBI/SIRF] Run ctest on SIRF demo scripts. (#723)
Evgueni Ovtchinnikov
Trying again to resolve issue #662.
Travis OSX builds fail if PET demos are run by ctest, apparently because of the Travis 10 min limit for running tests - without PET demos all builds succeed, but every OSX build spends more than 500 sec on SIRF tests.
You can view, comment on, or merge this pull request online at:
https://github.com/SyneRBI/SIRF/pull/723
Commit Summary
- reduced the number of PET demos to be run by ctest
File Changes
Patch Links:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Edoardo Pasca
I guess we could uncomment these lines and see what happens? The issue was closed and now a --noninteractive argument can be passed (skips matplotlib) but the demos never ran as part of the testing https://github.com/SyneRBI/SIRF/blob/e0f53f517bb4ef8f04550b6bdf24f7ead3977a9d/src/xSTIR/pSTIR/tests/CMakeLists.txt#L24-L26
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
@paskino pushed 1 commit.
- d9b4f92 Merge branch 'master' into ctest-pet-demos
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Casper da Costa-Luis
@casperdcl pushed 1 commit.
- 28b75eb reduced the number of PET demos to be run by ctest
You are receiving this because you are subscribed to this thread.
Casper da Costa-Luis
Merged #723 into master.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.