dcmjs-org / dcmjs

Javascript implementation of DICOM manipulation
https://dcmjs.netlify.com/
MIT License
291 stars 110 forks source link

feat(deflated): Added support for reading datasets with deflated transfer syntax #312

Closed PantelisGeorgiadis closed 1 year ago

PantelisGeorgiadis commented 1 year ago

Added support for reading datasets with Deflated Explicit VR Little Endian transfer syntax (1.2.840.10008.1.2.1.99). The deflated transfer syntaxe compress the whole dataset (except the file meta information header), using the standard “deflate” mechanism as used in gzip. In case the transfer syntax is recognized as Deflated Explicit VR Little Endian, the reading stream is "promoted" to a deflated reading stream, which internally uses pako to inflate the dataset bytes, prior reading them.

PantelisGeorgiadis commented 1 year ago

Thank you for your review @pieper! You can find the deflated sample file I used here. I would be grateful if you push the zip file to the data repo so I can update this PR.

pieper commented 1 year ago

Great - here's the release with the files as zip.

https://github.com/dcmjs-org/data/releases/tag/deflate-transfer-syntax

PantelisGeorgiadis commented 1 year ago

Thank you @pieper! I updated the tests to pull the deflated sample files from the data repo.

github-actions[bot] commented 1 year ago

:tada: This PR is included in version 0.28.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: