-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[Bug] PDF appear dark (wont properly load) #4118
Comments
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? |
@sedghi hey sedghi , i am not sure i understood you 100% , but i attach the dicom's meta data (censoring its patient info ) |
good morning, popping up due that it is still appears in the latest version |
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? |
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:
***@***.***>
|
No i meant the actual dicom file, do you have it and can you share it? |
|
So when i drag and drop it in local it works fine https://viewer-dev.ohif.org/localbasic What server are you using? Have you seen this |
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
Since there is not 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 |
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 |
@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 |
Interesting, I'll check |
Thanks for the response , after using v1beta1 looks like it's still returning undefined |
@sedghi |
I see the bulkDataURI in the 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 Maybe here https://docs.ohif.org/configuration/datasources/dicom-web/#bulkdatauri |
Done ! issue has been fixed , you can close this issue, Thanks! |
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
The text was updated successfully, but these errors were encountered: