Steve Pieper

Thanks for the report and the proposed fix 👍 Question though: shouldn't the VR be US or OW? https://dicom.innolitics.com/ciods/nm-image/voi-lut/00283010/00283006

That's a good question - I think the right answer is that whoever creates the element needs to provide the `_vrMap` like is done [here](https://github.com/dcmjs-org/dcmjs/blob/c209047861d2ef6c17526c6d1e29d5db7e895b29/src/adapters/DICOMMicroscopyViewer/MeasurementReport.js#L112-L114).

> Sorry for tagging you Steve, we meant to see if @dmlambo has any comment on this Thanks, I'm very happy for someone else to investigate this. 👍 For background:...

There may be people using this feature in the OHIF community so asking in the office hours makes sense.

> I think we should publish a 1.0 version of dcmjs without the adapters (as we took them outside) so that PRs similar to this happen to our external adapters...

> It started to return dataset.PatientName as [ { Alphabetic: 'BCM-3204-R3TG5-4709' } ] instead of regular text 'BCM-3204-R3TG5-4709' > This didn't seem right to me and I wanted to verify...

Thanks @wayfarer3130 > I think the plan should be to have a second branch for a half year or so to work on, then when that is stable, rename the...

Yes, this may need to be handled as a special case now. Perhaps by doing `JSON.stringify` on any PN?

Probably only a few transfer syntaxes were tested originally. I'm sure it would be possible to add support for that, but historically codecs haven't been easily dealt with. This is...

I'm not sure but I believe only uncompressed pixel data is currently supported.