search

cornerstonejs / dicomParser

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

There is a problem with the way the data is read #196

Closed timmy-li closed 1 year ago

timmy-li commented 2 years ago

In byteStream.js, function seek() does not notice the data size when jumping positions. This only limits it to no less than 0.

But after the data length is bigger than that, some methods still continue to read the wrong data.

Other functions also have similar problems.

timmy-li commented 2 years ago

image image

I get this information by inserting a method to get the information in parseDicomDataSet.js. At line 28, get {... .element, maxPosition}

Anyone have any ideas?

wangwendong-code commented 2 years ago

it is easy,wwdong726@163.com

yagni commented 2 years ago

@timmy-li It looks like the length of your pixel data (7fe0,0010) in your DICOM is incorrect. Can you confirm?

If so, the parser cannot be expected to read the file correctly, as it assumes the file is DICOM compliant.

yagni commented 1 year ago

Closing because stale