cornerstonejs / dicomParser

JavaScript parser for DICOM Part 10 data
MIT License
720 stars 230 forks source link

fix bug issue 105 #106

Closed xingbofeng closed 5 years ago

xingbofeng commented 6 years ago

fix the bug

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 51.885% when pulling 9f84a3c763443b10643383a288e15e322a72bd96 on xingbofeng:master into 00071e123d860ceacb7b5f82e4d84ff65b7dda6d on cornerstonejs:master.

yagni commented 5 years ago

Since DICOM character encoding is a lot more complicated, this PR may have worked for a specific use case but will fall apart for others. I'm going to close this because it'll introduce more problems. If you need something like this, please check out dicom-character-set.