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

[Bug] Green screen #3843

Closed
narbek-abd opened this issue Dec 7, 2023 · 8 comments
Closed

[Bug] Green screen #3843

narbek-abd opened this issue Dec 7, 2023 · 8 comments
Labels
fixed-in-dev-await-release This issue is fixed in master (viewer-dev) but we are pending testing for release (viewer.ohif.org)

Comments

@narbek-abd
Copy link

Describe the Bug

Hi.
Here is an archive.
e5824449-2d66de43-8459fe70-9ff5359c-97655c13.zip

After this commit 6651008 i see green screen for this study. In local mode it works, but I I use osimis/orthanc:22.11.3 and I downloaded source code before and after this commit and it was ok before this commit, so i don't think it is an issue with Orthanc

Steps to Reproduce

  1. use osimis/orthanc:22.11.3
  2. download source code before provided commit
  3. open the study

The current behavior

green screen

The expected behavior

proper viewer

OS

all

Node version

all

Browser

all

@narbek-abd narbek-abd added the Awaiting Reproduction Can we reproduce the reported bug? label Dec 7, 2023
@niirdan
Copy link

niirdan commented Dec 11, 2023

same issue here , using google.js configuration latest version .
there is an estimated time for a fix ?

@sedghi
Copy link
Member

sedghi commented Dec 13, 2023

I don't think it is related to that commit honestly. This is a long lasting bug in Orthanc that we tried to communicate to fix, but they fixed and later reverted the fix. https://discourse.orthanc-server.org/t/orthanc-convert-ybr-to-rgb-but-does-not-change-metadata/3533/16

@sedghi
Copy link
Member

sedghi commented Dec 13, 2023

@niirdan
Copy link

niirdan commented Dec 14, 2023

hey @sedghi thanks for your answer, this is very appreciated ! how does orthanc works inside ohif ? is it a remote server that the ohif sends a request or this is a static node module , because if it is a node module , maybe the temporary solution is to downgrade it to the version with the fix that they released ?

@sedghi
Copy link
Member

sedghi commented Dec 19, 2023

seems like this will help

https://discourse.orthanc-server.org/t/orthanc-convert-ybr-to-rgb-but-does-not-change-metadata/3533/20?u=alireza

@niirdan
Copy link

niirdan commented Dec 21, 2023

seems like this will help

https://discourse.orthanc-server.org/t/orthanc-convert-ybr-to-rgb-but-does-not-change-metadata/3533/20?u=alireza
@sedghi
this actually fix the issue !!! many thanks for mentioning!
please consider editing google.js in the next branch with this fix

@sedghi
Copy link
Member

sedghi commented Jan 9, 2024

Checkout lastest master branch, it should have been fixed with the recent push

@sedghi sedghi added the fixed-in-dev-await-release This issue is fixed in master (viewer-dev) but we are pending testing for release (viewer.ohif.org) label Jan 9, 2024
@sedghi
Copy link
Member

sedghi commented May 1, 2024

We just release the OHIF 3.8, you can find more details here https://ohif.org/release-notes/3p8/
If you still encounter this issue in 3.8, please re-open this.

@sedghi sedghi closed this as completed May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-in-dev-await-release This issue is fixed in master (viewer-dev) but we are pending testing for release (viewer.ohif.org)
Projects
None yet
Development

No branches or pull requests

3 participants