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

Jump to bottom

All tests using --no-main-window fail due to crash on exit #4541

Closed
slicerbot opened this issue Mar 13, 2020 · 4 comments
Closed

All tests using --no-main-window fail due to crash on exit #4541

slicerbot opened this issue Mar 13, 2020 · 4 comments
Assignees
@cpinter
Milestone
Slicer 4.13

Comments

@slicerbot
Copy link
Collaborator

This issue was created automatically from an original Mantis Issue. Further discussion may take place here.
@slicerbot slicerbot added this to the Slicer 4.11.0 milestone Mar 13, 2020
@lassoan
Copy link
Contributor

lassoan commented Mar 26, 2020

@cpinter is this still an issue?

@jamesobutler
Copy link
Contributor

The tests mentioned in the mantis issue are not failing anymore as was originally described.

@cpinter
Copy link
Member

cpinter commented May 28, 2020

Let me try the tests mentioned in the Mantis issue again. Many of the SlicerRT tests failed because the LandmarkRegistration module was missing. Now that it's back I'll run the tests again.

@cpinter
Copy link
Member

cpinter commented May 28, 2020

There is no crash any more.

@cpinter cpinter closed this as completed May 28, 2020
cpinter added a commit to SlicerRt/SlicerRT that referenced this issue May 28, 2020
@cpinter
BUG: Fix DicomRtImportTest
7f56d8e 
- The baseline numbers changed but the test was not updated
  - The rest of the python tests need to be fixed as well!
- Reinstated --no-main-window as there is no crash any more when enabled, see Slicer/Slicer#4541
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cpinter cpinter

Labels
None yet
Milestone
Slicer 4.13
Development

No branches or pull requests
4 participants
@lassoan @cpinter @slicerbot @jamesobutler