Jon Haitz Legarreta Gorroño

Results 128 comments of Jon Haitz Legarreta Gorroño

@ChristianJacobsen contributions are always welcome :100:, and as @dzenanz says, this is still relevant. It would be awesome to finally have all necessary overloads in ITK ! So thanks for...

@ChristianJacobsen Another good candidates to get this enhancement are these two: https://github.com/InsightSoftwareConsortium/ITK/blob/f81578bbb6242032b611d7e3e4dca3b7557fbbe0/Modules/Core/Common/include/itkNeighborhood.hxx#L170 `m_StrideTable` and `m_OffsetTable`. Or the `m_MembershipFunctions` vector here: https://github.com/InsightSoftwareConsortium/ITK/blob/047fa6934ac32711d1d7b84fa6eb81c72db01960/Modules/Segmentation/Classifiers/include/itkClassifierBase.hxx#L51 One would call `itkPrintSelfObjectMacro(MemberThanCanBeNull)` for objects that can be...

@ChristianJacobsen no worries. Take your time. Thanks.

Not sure about what the errors imply: https://open.cdash.org/viewBuildError.php?buildid=7260537 Suggestions are welcome.

Push forced to see the error reports back and see where we were at this level.

Thanks for the comment @N-Dekker . Then, as pointed in [this comment of yours](https://github.com/InsightSoftwareConsortium/ITK/pull/2568#discussion_r645549336) I'll mark it to `status:Blocked` until the proposed `itkGetConstNonVirtualMacro` gets added to the ITK code base.

10fce06 might go somehow unnoticed inside the PR and unless one takes a look at the commit history, but did not want to push a PR that added code that...

> @jhlegarreta Is this still something that needs to be worked on? If so, could you please rebase and make a game plan for what needs to be done? Thanks...

@jhlegarreta > - I did not want to change the name of the current macros to contain `virtual` in their names. This affects virtually all header files. My proposal is...