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

Issue with Zoom behavior on SLIM in Production tier for TCGA image slides #63

Open
pgundluru opened this issue Jun 15, 2022 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@pgundluru
Copy link

@dclunie @wlongabaugh has identified an issue with straight Zoom in SM viewer on 6/15/2022

(Slide: TCGA-OR-A5J2-01A-01-TS1)

Zooming straight in (no panning) gives inconsistent zoom and black boxes in image

https://viewer.imaging.datacommons.cancer.gov/slim/studies/2.25.196877316647058549027120300438839406015/series/1.3.6.1.4.1.5962.99.1.3129682471.1589528748.1639512255015.2.0

There are 2 series associated with this case ID and the other one doesn't have this particular issue so it may be a isolated issue with the particular slide
[https://viewer.imaging.datacommons.cancer.gov/slim/studies/2.25.1968773166470585490271203004388[…]1.5962.99.1.3144471409.451958541.1639527043953.2.0]

image

@pgundluru pgundluru added the bug Something isn't working label Jun 15, 2022
@dclunie
Copy link

dclunie commented Jun 15, 2022

I could not reproduce this behavior -the same slide zooms and pans fine for me.

@dclunie
Copy link

dclunie commented Jun 15, 2022

Actually I withdraw that comment - I zoomed too fast - there seems to be a problem with an intermediate layer - I will investigate

@dclunie
Copy link

dclunie commented Jun 15, 2022

The old conversion logs suggest this is one that had an issue that may be related to the "empty tile" SVS issue, which should be addressed by the upcoming reconversion - this slide should be a good test case.

@fedorov
Copy link
Member

fedorov commented Jun 16, 2022

This is duplicate of ImagingDataCommons/slim#97, will keep open until confirm with the re-converted data.

@hackermd
Copy link
Contributor

Thanks for investigating @dclunie.

@dclunie
Copy link

dclunie commented Jun 27, 2022

Yes, I did too, thanks ... David

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants