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

[HELP - BUG] Custom Data source Viewer not rendering - black blank screen #4161

Open
highoncarbs opened this issue May 23, 2024 · 3 comments
Labels
Awaiting Reproduction Can we reproduce the reported bug?

Comments

@highoncarbs
Copy link

Describe the Bug

I wanted the viewer to be accessible like this /vewier/scan?id=6sdasd838878sad

So I created a new datasource in /extensions/default/src/ScanIdDataSource/index.js , which is basically a clone of DicomJSONDataSource updated to fetch the id param and query an API, this returns dicomJSON.

When I go to this route, the console prints the statements ScanIdDataSource/index.js, but nothing renders on the screen and no console errors.

Steps to Reproduce

This is the script /extensions/default/src/ScanIdDataSource/index.js that is printing the console logs
image

Added to datasources
image

Browser View
image

Console
image

The current behavior

Blank Viewer

The expected behavior

It should load the scans in the Viewer

OS

Windows 11

Node version

20

Browser

Latest Chrome and edge

@highoncarbs highoncarbs added the Awaiting Reproduction Can we reproduce the reported bug? label May 23, 2024
@highoncarbs
Copy link
Author

Alright , I think this issue is with v3.9, it's working on v3.5

@highoncarbs
Copy link
Author

Ok, so I made it to route correctly on 3.8, but it is throwing an error on my customDatasource

TypeError: dataSource.getStudyInstanceUIDs is not a function

And the same data works in dicomJSON @sedghi , would love to get your quick help on this

@highoncarbs highoncarbs changed the title Custom Data source Viewer not rendering - black blank screen [HELP - BUG] Custom Data source Viewer not rendering - black blank screen May 30, 2024
@sedghi
Copy link
Member

sedghi commented Jun 6, 2024

can you implement getStudyInstanceUIDs maybe? take a look at how other data sources implement it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Reproduction Can we reproduce the reported bug?
Projects
None yet
Development

No branches or pull requests

2 participants