-
Notifications
You must be signed in to change notification settings - Fork 23
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
ENH: Bump elastix for ITK_MODULE_TEMPLATE additions #102
Conversation
elastix develop branch, 2021-02-12.
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.
The notebook tests suddenly fail on this branch, due to an itk import error. A similar failure occurs on the #101 PR. I suppose the errors occur due to an erroneous build on dockerhub (v0.10.0), will one of these PRs fix these issues as well?
These failures appear to be related to a lack of 4D support in the ITK package -- which indicates it is 5.1 instead of 5.2. I will also bump the versions in the Binder requirements.txt to address this. |
@N-Dekker we are observing a few small changes in the result on Linux: https://open.cdash.org/test/342115602 Any ideas why? Since it essentially the same result, I will increase the image comparison. tolerance. |
Account for slight variations in the result on Linux between elastix versions.
I only excluded the Now I wonder: are all elastix classes now indeed properly exported? |
We made progress on the exports, but there are still a lot of missing https://github.com/SuperElastix/elastix/blob/develop/Common/ImageSamplers/itkImageGridSampler.h While the exports could be addressed in the future, the patch will prevent crashing on macOS for now: |
Thanks for asking, Matt. I'm not sure but we started using |
Thanks for the note, Neils. Yes, that would make sense. |
@N-Dekker we are getting the build error with the OpenCL builds:
Does this ring a bell? |
If this branch can point to the most recent elastix update . The MacOS issue will at least be fixed wrt the segfault and issue #52 can finally be closed :). |
To address macOS crashes.
7c24311
to
f6714e2
Compare
Honestly no. Can you give me another clue, please? So the compilation of various |
f6714e2
to
e13b306
Compare
1667f7a
to
4045f0c
Compare
@N-Dekker with the update to ITK 5.2, the manylinux2014 compiler we no longer have the error. |
elastix develop branch, 2021-02-12.