Dženan Zukić

Results 653 comments of Dženan Zukić

It would be good to merge this.

I think tag `v5.1.2` should work too, as it contains [MinPriorityQueueElementWrapper fix](https://github.com/InsightSoftwareConsortium/ITK/commit/b02778025970a8a7e68642c81e5049afb5139748). If not, https://github.com/InsightSoftwareConsortium/ITK/commit/facce8fcf63a31215df82b636ba7743ddcc63c4b is what I have in one of the builds.

Option 1 (utf-8) seems more logical. It will require a bit of extra coding for Windows, see [SO 30829364](https://stackoverflow.com/questions/30829364/open-utf8-encoded-filename-in-c-windows).

Is this issue addressed by one of your recent PRs? Perhaps write `Closes #66` in the commit message or PR description to reference this issue and have it automatically closed...

`vtksys` and `itksys` are a rename of [kwsys](https://gitlab.kitware.com/utils/kwsys). So within VTK, KWSys is compiled under the name `vtksys`. Library content remains the same.

> In the meantime, copying the removed file into src and adding it as a _h file to be configured, in the cmakelists.txt file, works. I am giving this a...

`itkStatisticsUniqueLabelMapFilterTest` sounds familiar, because I think it is flaky. So this might be a genuine bug discovery. We shouldn't sweep it under the carpet. Does anyone have time to tackle...

It looks like pleasing the CI will take some doing.