Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retrieve binary DICOM instance #736

Closed
MathisGuilhin opened this issue Jul 30, 2019 · 8 comments
Closed

Retrieve binary DICOM instance #736

MathisGuilhin opened this issue Jul 30, 2019 · 8 comments

Comments

@MathisGuilhin
Copy link

Hello OHIF team,

According to DICOM WADO-RS specifications, we should be able to retrieve binary DICOM instances from the PACS.
But where is it actually manipulated? Where can i retrieve this instance (that I would like to save)?

@swederik
Copy link
Member

When we use WADO-RS we are retrieving the pixel data and the metadata separately. First we do a Study Metadata call and then we grab the pixel data for each frame.

If you want the entire instance you'll have to do the call separately or use WADO-URI.

@MathisGuilhin
Copy link
Author

With the DICOMWeb-client api, I managed to retrieve DICOM binary instance with this method:

https://github.com/dcmjs-org/dicomweb-client/blob/5c1a0a0138b0a77db661df84a6ac80737668b7e1/src/api.js#L1121

But when I use retrieveSeries or retrieveStudy, only the first instance of DICOM is readable.
Besides, is there a way to save a whole series in one DICOM File?

Thank you!

@swederik
Copy link
Member

Hmm that could be a bug with retrieveSeries and/or retrieveStudy, I did change those recently.

You can't get them in one file, retrieveSeries/Study with dicomweb-client should give you back a list of instances as arraybuffers though. If it's not doing that, we should create an issue there.

@MathisGuilhin
Copy link
Author

It actually gives back a list of instances as arraybuffers, but only the first one is readable (or maybe im not doing it right).
A loop with instance should do the trick here, but it's still weird it isn't working though

@dannyrb
Copy link
Member

dannyrb commented Aug 26, 2019

@MathisGuilhin have you made any progress on this? Anything we can do to assist?

@MathisGuilhin
Copy link
Author

No, As I said I was only able to retrieve DICOM instances, but I haven't tried with all the recent commits you've made since

@swederik
Copy link
Member

swederik commented Sep 2, 2019

We should move this to dicomweb-client then. That sounds like an issue on that library.

@dannyrb
Copy link
Member

dannyrb commented Sep 6, 2019

@dannyrb dannyrb closed this as completed Sep 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants