Dženan Zukić

Results 653 comments of Dženan Zukić

I just noticed [compile errors](https://open.cdash.org/viewBuildError.php?buildid=6018132) in non-default module `ITKFEMRegistration`: `C:\Dashboard\ITK\Modules\Registration\FEM\include\itkFEMRegistrationFilter.h(166): error C2061: syntax error: identifier 'VectorInterpolateImageFunction' [C:\Dashboard\ITK-build\Modules\Registration\FEM\test\ITKFEMRegistrationTestDriver.vcxproj]`. Is this on your radar @blowekamp?

Failures started on 2019-07-08. The probable culprit is 9ef036a9cbc2a9f42059f4574fdcf4f0fc83bf88, by @maekclena.

There has been work on this over the years. It is possible there is only one left. @blowekamp Maybe that one was left intentionally? If so, we could close this...

I don't know what is the name of the CMake target, so I would need `include_directories`. The proposed solution should also have a side benefit of showing these header files...

I like it, but there is a lot work there. We should strive to keep the functionality on the same level, including Python accessibility. Also, there are a few examples...

How is it expected to interact with [SpatialOrientationAdapter](https://github.com/InsightSoftwareConsortium/ITK/blob/master/Modules/Core/Common/include/itkSpatialOrientationAdapter.h)? What about [this test](https://github.com/InsightSoftwareConsortium/ITK/blob/master/Modules/Core/Common/wrapping/test/itkSpatialOrientationAdapterTest.py)? I assume that [DICOMOrientation](https://github.com/InsightSoftwareConsortium/ITKSimpleITKFilters/blob/master/include/itkDICOMOrientation.h) would replace [SpatialOrientation](https://github.com/InsightSoftwareConsortium/ITK/blob/a3d3e53d685b43cf9d07602a8ce4651f2bfc4680/Modules/Core/Common/include/itkSpatialOrientation.h#L46)? Namely, as long as we can accomplish things like [these](https://github.com/InsightSoftwareConsortium/GetYourBrainTogether/blob/main/HCK01_2022_Virtual/Tutorials/MetadataPreservation/Orient_filter.ipynb) using...

Going forward, I should have more time to devote to code reviews, and I should be able to provide early feedback on this effort - should you want it, of...

Is this ready for merging? Making the method more general could happen in a follow-up PR, if needed.

@blowekamp @issakomi @seanm is anyone brave enough to update bundled version of TIFF? It seems like we have some very old version of it.