bug: switch to integer for the skip flag

QIICR / dcmqi

dcmqi (DICOM for Quantitative Imaging) is a free, open source C++ library for conversion between imaging research formats and the standard DICOM representation for image analysis results

https://qiicr.gitbook.io/dcmqi-guide/

BSD 3-Clause "New" or "Revised" License

234 stars 62 forks source link

bug: switch to integer for the skip flag #499

Closed fedorov closed 3 months ago

fedorov commented 3 months ago

re #498

It appears that defaults are ignored for boolean SEM parameters. Even though the parameter was set to true in https://github.com/QIICR/dcmqi/blob/2990dbd01d1fa61c16777950732ff19dc87318e3/apps/seg/itkimage2segimage.xml#L58-L65, the default reported by the executable was 0.

Local tests show that integer parameters do not have this problem.

Also switch to the most recent has for SlicerExecutionModel.