Dženan Zukić

icon indicating a website link https://www.kitware.com/dzenan-zukic/ icon indicating an email [email protected]

icon company @KitwareMedical icon location Carrboro, NC, USA icon location Staff R&D Engineer @ Kitware. Dealing with image analysis, deep learning, 3D space manipulation, surgical guidance, etc.

Results 237 comments of Dženan Zukić

Remove MetaProgrammingLibrary and replace by C++14 STL equivalents

All the involved files are in https://github.com/InsightSoftwareConsortium/ITK/tree/master/Modules/Core/Common/include: - itkMetaProgrammingLibrary.h - itkIsNumber.h - itkIsConvertible.h - itkVariableLengthVector.h A work-in-progress branch is in my repository: https://github.com/dzenanz/ITK/commits/cmake320policies.

NrrdImageIO writes incorrect file when system locale uses comma instead of dot for decimal numbers (space directions)

This is similar to issue #2067, and I expect the solution (#2297) to be similar too. @rAum can you propose a PR with the fix?

itk(GDCM|DCMTK)SeriesFileNames: memory leaks, bad behavior

I don't think that memory leaks are known. @malaterre @issakomi correct me if I am wrong. @PHLF do you have a simple repro case for a memory leak?

ENH: Register/UnRegister only in SmartPointer Con/Destructor.

I think that ITK 6.0 is not that far off. Maybe 2-3 years away? @thewtex could clarify the plan, but it also depends on the sentiment of the rest of...

ENH: Register/UnRegister only in SmartPointer Con/Destructor.

@malaterre should be able to answer the question about `Modules/ThirdParty/GDCM/src/gdcm/Source/`.

Windows for ARM support

What are the errors? Ideally, set up a [nightly build](https://itk.org/Wiki/ITK/Policy_and_Procedure_for_Adding_Dashboards) using your machine so we are aware of possible build errors and warnings.

Fix and update all remote modules to adhere to new testing structure

Shouldn't we disable and delete the current configuration after we make the new configuration work? So move current steps 1 and 2 to be after current step 6. As @thewtex...

Fix and update all remote modules to adhere to new testing structure

It is good to hear that remotes will have another round of updates! Thanks for working on this @mseng10.

Fix and update all remote modules to adhere to new testing structure

> it seems quite different from the one in ITKModuleTemplate Harmonizing things is welcome! But do not confuse [module template's own CI](https://github.com/InsightSoftwareConsortium/ITKModuleTemplate/blob/master/.github/workflows/build-test-package.yml) with the [generated project's CI](https://github.com/InsightSoftwareConsortium/ITKModuleTemplate/blob/master/%7B%7Bcookiecutter.project_name%7D%7D/.github/workflows/build-test-package.yml).

Fix and update all remote modules to adhere to new testing structure

Module template also needs to invoke the cookie cutter generation step, which should not be done in regular modules. The [generated project's CI](https://github.com/InsightSoftwareConsortium/ITKModuleTemplate/blob/master/%7B%7Bcookiecutter.project_name%7D%7D/.github/workflows/build-test-package.yml) should be mostly the same as regular...