[Bug] PDF appear dark (wont properly load)

[niirdan]

Describe the Bug

Looks like pdf couldn't be loaded properly

Steps to Reproduce

open a study containing pdf document

*note! i am using google healthcare api .

The current behavior

pdf viewer inside ohif wont load

The expected behavior

pdf presented correctly

OS

window 11

Node version

v18.17.1

Browser

latest chrome update

[sedghi]

What is your server? can you provide the anonymized data?

Could you kindly provide the data if it has been anonymized and you can confirm that there is no patient health information present in any of the headers or embedded within the pixel data?

[niirdan]

@sedghi hey sedghi , i am not sure i understood you 100% , but i attach the dicom's meta data (censoring its patient info )
I am using dcmtk for encapsulating the pdf to dicom and dcmtk dcmodify to add necessary information about the study and patient

[niirdan]

good morning, popping up due that it is still appears in the latest version

[sedghi]

Could you kindly provide the data if it has been anonymized and you can confirm that there is no patient health information present in any of the headers or embedded within the pixel data?

[niirdan]

Hey alizera thanks for your response, I uploaded a photo of the meta data’s file (pdf) I confirm it doesn’t contain real patient information- it has been censored

…

On Tue, 28 May 2024 at 18:42 Alireza ***@***.***> wrote: Could you kindly provide the data if it has been anonymized and you can confirm that there is no patient health information present in any of the headers or embedded within the pixel data? — Reply to this email directly, view it on GitHub <#4118 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A5B2UZTXLZUCQD3ZEZNKICTZESQWNAVCNFSM6AAAAABHSST46OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZVGU3TEMRYGE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[sedghi]

No i meant the actual dicom file, do you have it and can you share it?

[niirdan]

No i meant the actual dicom file, do you have it and can you share it?
Yes Of course !
unfortunately github doesn't allow dcm format so i will upload it here :
https://www.jumbomail.me/l/en/gallery/697273696E617777705662464B3342347436653838673D3D/1181030228
let me know if you have an issue downloading it

[sedghi]

So when i drag and drop it in local it works fine https://viewer-dev.ohif.org/localbasic

What server are you using?
these things are most likely server depndent

Have you seen this
https://docs.ohif.org/configuration/datasources/dicom-web/#singlepart

[sedghi]

I saw you use google healthcare

I uploaded your data to our google healthcare and i see the instance is this

And basically this comes back undefined

 const pdfUrl = dataSource.retrieve.directURL({
      instance,
      tag: 'EncapsulatedDocument',
      defaultType: MIMETypeOfEncapsulatedDocument || 'application/pdf',
      singlepart: 'pdf',
    });

Since there is not EncapsulatedDocument tag that google sends back and as I understand this is a type 1 attribute

https://dicom.innolitics.com/ciods/encapsulated-pdf/encapsulated-document/00420011

Our demo server (static dicom web) provides a bulkDataURI

And Orthanc provides this too

So i believe this is a issue with how google is responding back.

Tagging @jasonklotzer to see if he knows more about DICOM PDF on Google Healthcare

[jasonklotzer]

BulkDataURI is only available when using the beta API right now. Please try it out and tell me how it goes.

[niirdan]

BulkDataURI is only available when using the beta API right now. Please try it out and tell me how it goes.

Hey jason @jasonklotzer , thanks for responding
do to mean to change this by adding beta1 to the version section of this url ? (v1beta1) ?
'https://healthcare.googleapis.com/v1/projects/XXXX/locations/me-west1/datasets/XX-dataset/dicomStores/XX-datastore/dicomWeb' on ohif's google.js settings ?
as far as i understand it still return undefined due to the fact that the file's meta data haven't changed ..
I will be happy for a tiny explanation what to change in this process ?

[jasonklotzer]

@sedghi It would be helpful for you to take a look at the API and validate whether that conforms to the OHIF expectations for interacting with a BulkDataURI provider?

@niirdan Yes, the Google Healthcare API DICOM store added support for generating BulkDataURI's and retrieving their referenced payload, and that support is currently in the v1beta1 version of the API. The underlying database has also been populated (backfilled) with any relevant bulk data references, so upon using this version of the API you should see BulkDataURI references.

[sedghi]

Interesting, I'll check

[niirdan]

@sedghi It would be helpful for you to take a look at the API and validate whether that conforms to the OHIF expectations for interacting with a BulkDataURI provider?

@niirdan Yes, the Google Healthcare API DICOM store added support for generating BulkDataURI's and retrieving their referenced payload, and that support is currently in the v1beta1 version of the API. The underlying database has also been populated (backfilled) with any relevant bulk data references, so upon using this version of the API you should see BulkDataURI references.

Thanks for the response , after using v1beta1 looks like it's still returning undefined

[niirdan]

@sedghi
Hey Alizera , just to make sure,
does it appear to be a bug in ohif ? or it seems to be a bad behavior of google's api ?
I tried using the beta version of the api as @jasonklotzer told , and it is still seems to appear dark .

[sedghi]

I see the bulkDataURI in the v1beta1

and i was able to render it

these are the changes required

Note the bulkDataURI enabled

@niirdan Would you be able to add a section to our documentation with a pull request describing the v1beta1 and bulkDataURI?

Maybe here https://docs.ohif.org/configuration/datasources/dicom-web/#bulkdatauri

[niirdan]

I see the bulkDataURI in the v1beta1

and i was able to render it

these are the changes required

Note the bulkDataURI enabled

@niirdan Would you be able to add a section to our documentation with a pull request describing the v1beta1 and bulkDataURI?

Maybe here https://docs.ohif.org/configuration/datasources/dicom-web/#bulkdatauri

Done ! issue has been fixed , you can close this issue, Thanks!