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

Run ctest on SIRF demo scripts. #723

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

evgueni-ovtchinnikov
Copy link
Contributor

@evgueni-ovtchinnikov evgueni-ovtchinnikov commented Jun 19, 2020

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.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@paskino
Copy link
Contributor

paskino commented Feb 4, 2025

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

#add_test(NAME PET_DEMOS_PYTHON
# COMMAND ${PYTHON_EXECUTABLE} run_all.py --non-interactive
# WORKING_DIRECTORY "${CMAKE_SOURCE_DIR}/examples/Python/PET")

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

Successfully merging this pull request may close these issues.

None yet

2 participants