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

Compiler error concerning ITK_THREAD_RETURN_VALUE #20

Open
jhlegarreta opened this issue Oct 2, 2018 · 2 comments
Open

Compiler error concerning ITK_THREAD_RETURN_VALUE #20

jhlegarreta opened this issue Oct 2, 2018 · 2 comments

Comments

@jhlegarreta
Copy link
Member

jhlegarreta commented Oct 2, 2018

There seems to be a compiler error involving the ITK_THREAD_RETURN_VALUE define:

Error C2065 'ITK_THREAD_RETURN_VALUE': undeclared identifier itkCannyEdgeDetectionImageFilter2  
\lesionsizingtoolkit\include\itkCannyEdgeDetectionRecursiveGaussianImageFilter.hxx 305

@dzenanz could you please cast some light on this?

Also missing to use the ProgressTransformer class instead of ProgressReporter.

@dzenanz
Copy link
Member

dzenanz commented 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
Copy link
Member Author

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.

This was referenced Oct 11, 2018
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

No branches or pull requests

2 participants