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
Fix unit_testing/test_labels.py::test_remove_missing_labels #3191
Conversation
571a24d
to
7e6b511
Compare
Amazing, thank you for taking this on. I'll take a look momentarily. |
Woops looks like I broke something. Will look. |
….value directly instead of relying on membership check with "in" operator.
7e6b511
to
5160e3d
Compare
CI failed because wsl couldn't be installed on windows (https://travis-ci.com/github/neuropoly/spinalcordtoolbox/jobs/475181141#L45)... |
- don't compare all voxels, only labels - spotcheck the relevant labels - use actual labels as test param instead of image
5160e3d
to
4b62fab
Compare
ping @kousu |
@Drulex I can review as well, if that's alright? 🙂 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is fixing 2 issues, right?
- Broken test (because of wrong input data)
- Slow, inefficient test
If so, it might be good to clarify the issue description, as #3153 only reports 1).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for clarifying! LGTM.
Checklist
GitHub
PR contents
Description
Closes #3153