David Giese

icon indicating a website link innolitics.com icon indicating an email [email protected]

icon company @innolitics icon location New York City icon location I co-founded a medical device software and regulatory firm. I enjoy coding, medical devices, and improving the medical device industry through great software.

Results 55 comments of David Giese

Enable a force write option for files with abnormal VRs

Hey @pieper, I hope you're doing well. In addition to the ability to write an invalid file, we also were hoping validate dcmjs DicomDicts somehow. Can you confirm that neither...

Enable a force write option for files with abnormal VRs

Thanks @pieper , we'll checkout the `allowInvalidVRLength` option. I was actually thinking of just basic VR validation, but something more sophisticated like [http://dclunie.com/dicom3tools/dciodvfy.html](dciodvfy) would be even better. I think for...

Enable a force write option for files with abnormal VRs

@ejcurtis I'm pretty sure it doesn't, but I'd check in the editor as well as with Russell.

Enable a force write option for files with abnormal VRs

We’re busy with other priorities at the moment. I’m not sure when we’ll get to this. > On Mar 3, 2021, at 5:29 PM, Woonchan Cho wrote: > > ...

switching off extrapolation

There is not currently a graceful way to do this. In our demo we manually do this by setting values that are `nan` in the linear interpolation to `nan`: https://github.com/innolitics/natural-neighbor-interpolation/blob/master/demo/demo.py#L45...

ENH proposals from contrib-pydicom: read a folder, use proper dtype for rescaled images

@jond01 thank you for the thoughtful suggestions. I agree that (1) would be a useful function to add. I also agree that we don't want to force users to convert...

Parse Enumerated Values and Defined Terms

FYI @ccwoolfolk here is another, small-sized issue that could be useful/relevant. Don't feel any need to re-prioritize though unless you think it makes sense.

'Unable to parse error' on SR document, even though it opens in other DICOM viewers

Thank you for the kind words! We'll take a look at this bug once we have a break from client work.

File Editor (beta) labels Referenced Frame of Reference Sequence incorrectly

Thanks a lot @TheGypsyCoder for the bug report. If at all possible, could you share a de-identified version of the DICOM file that triggered the error?

File Editor (beta) labels Referenced Frame of Reference Sequence incorrectly

Fantastic, thank you Gary! We'll be sure to let you know what we find when look into this.