You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
LesionSizingToolkit has its own version of Canny filter. This filter has not been updated to use the new threading model. I don't know what the difference was (if any) when compared to normal Canny filter. Anyway, this filter will have to be rewritten to inherit from the original Canny, or updated in a similar way to original Canny. Canny filter underwent all the standard refactoring, plus this specific simplification.
You are right, the module would need some work to update it, adapt to the changes since 6 years now, avoid duplicates, and eventually get its way into the main toolkit.
At this time, I'll just do the minimal changes to make it build against ITK v5. I'll add a WIP branch with CI, and once that is completed, we will be a in a better position to focus on further improvements.
The compile error is gone: itkCannyEdgeDetectionRecursiveGaussianImageFilter.hxx now returns ITK_THREAD_RETURN_DEFAULT_VALUE, and the module builds and passes its tests against ITK 5.4.6 and ITK 6 main on Linux, macOS, and Windows in #61. The filter still uses the pre-ITK5 threading model (ITK_THREAD_RETURN_TYPE callbacks and ProgressReporter instead of ProgressTransformer); that modernization is tracked under the refactor issue #22.
Activity
dzenanz commentedon Oct 2, 2018
LesionSizingToolkit has its own version of Canny filter. This filter has not been updated to use the new threading model. I don't know what the difference was (if any) when compared to normal Canny filter. Anyway, this filter will have to be rewritten to inherit from the original Canny, or updated in a similar way to original Canny. Canny filter underwent all the standard refactoring, plus this specific simplification.
jhlegarreta commentedon Oct 3, 2018
You are right, the module would need some work to update it, adapt to the changes since 6 years now, avoid duplicates, and eventually get its way into the main toolkit.
At this time, I'll just do the minimal changes to make it build against ITK v5. I'll add a WIP branch with CI, and once that is completed, we will be a in a better position to focus on further improvements.
hjmjohnson commentedon Oct 8, 2026
The compile error is gone:
itkCannyEdgeDetectionRecursiveGaussianImageFilter.hxxnow returnsITK_THREAD_RETURN_DEFAULT_VALUE, and the module builds and passes its tests against ITK 5.4.6 and ITK 6mainon Linux, macOS, and Windows in #61. The filter still uses the pre-ITK5 threading model (ITK_THREAD_RETURN_TYPEcallbacks andProgressReporterinstead ofProgressTransformer); that modernization is tracked under the refactor issue #22.