This is a quick fix for #294, by returning an error in the case that the dicom indicates signed pixel data and a negative pixel data value is detected.
If users are passing the SkipProcessingPixelDataValue or SkipPixelData option, this will bypass this error since this will only happen when trying to parse the raw Native PixelData.
It would be useful to add more tests for other bit widths, and some of the tests can be better structured (which can happen in follow ups), but I'd like to get this in sooner rather than later. We can follow up on getting signed integers fully supported directly.
This is a quick fix for #294, by returning an error in the case that the dicom indicates signed pixel data and a negative pixel data value is detected.
If users are passing the
SkipProcessingPixelDataValue
orSkipPixelData
option, this will bypass this error since this will only happen when trying to parse the raw Native PixelData.It would be useful to add more tests for other bit widths, and some of the tests can be better structured (which can happen in follow ups), but I'd like to get this in sooner rather than later. We can follow up on getting signed integers fully supported directly.