Bill Wallace comments

Results 50 comments of


                                            Bill Wallace

fix: Crash/hang issues and a minor log issue

No, if they set it specifically to 10 then it can stay as 10. On Thu, 5 May 2022 at 11:42, Alireza ***@***.***> wrote: > ***@***.**** commented on this pull...

fix(OHIFv3):Allow multiple study UIDs in the path.

@sedghi - can you take a quick look, very simple fix.

feat(syncGroups):Add registration for new synchronizers

This change can be committed independently now from the hanging protocol change, but I would have to extract the HP stuff first, so that this one can go on top...

Add support to Stratovan DICOS file which is based on DICOM format

You will have to explain where the software is incompatible. It SHOULD read DICOS files, although I'm definitely not very familiar with them to know. Please attach a test case...

TypeError: Cannot redefine property: Alphabetic

Will look as soon as the test is available.

TypeError: Cannot redefine property: Alphabetic

Will try to get a PR for this by the end of the day - I have a fix for it already, just need to add a unit test. I...

TypeError: Cannot redefine property: Alphabetic

I have a PR for this right now, but the issue exposes the fact that the normalized data refers to the original data, so modifying it will modify the original/underlying...

fix: images with color pallet lut not displaying properly

Please just use Array.isArray, drop the two extra files, and add a unit test.

Proof of concept decoding HTJ2K with progressive loading and rendering

Generally LGTM - if we use a custom TSUID. Hopefully Erik can take a look as I'm not terribly familiar with the details of exactly where the codec is created.

BulkDataURI response should handle non-multipart as per the DICOMweb standard

@pieper - can I have your thoughts on implementing this one? I was going to modify my local copy to handle either response type and submit a PR for it.