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

Mobile viewer crash with big number of images on Google Chrome #255

Closed
ftaurino opened this issue Jul 25, 2016 · 7 comments
Closed

Mobile viewer crash with big number of images on Google Chrome #255

ftaurino opened this issue Jul 25, 2016 · 7 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists wontfix This will not be worked on
Milestone

Comments

@ftaurino
Copy link

Hi all,

I've a mobile viewer, version 0.15-beta, which can display CT with over 200 images with no glitches.
But when I try to load a CT with over 400 images, the Google Crome (version 52) tab crashes.
Firefox 47 is slower than Chrome in rendering the two series, but it does not crash.
The Mozilla browser also correctly display the spinning circle while receiving the images...

Francesco

@ivmartel ivmartel added the bug Something isn't working label Aug 16, 2016
@ivmartel ivmartel added this to the 0.16.0 milestone Aug 16, 2016
@ivmartel
Copy link
Owner

Kind of similar to #236, you are reaching the maximum the browser can handle. I need to find a better way of handling memory.

@Zaid-Safadi
Copy link

One way might be to support virtual loading of images. such that you load only enough for current view + X (where X is a number of adjacent images) and on scroll you start pre-loading and freeing. Another way would be loading the full stack in low resolution e.g. 128x128 (if possible) as a preview and when user release the scroll start loading the current full resolution image, this might require a server that support fetching low resolution though.

@ajabnoor
Copy link

this can be a good solution Zaid

@ivmartel ivmartel modified the milestones: 0.18.0, 0.17.0 Dec 18, 2016
@ivmartel ivmartel modified the milestones: 0.18.0, 0.19.0 Mar 26, 2017
@ivmartel ivmartel modified the milestones: 0.19.0, 0.20.0 Apr 11, 2017
@vmaindarkar
Copy link

I handled the big data but having one issue. In some case Tool.info.position count start from zero and in some case it's shows max count. Can u help how it's come

@ivmartel
Copy link
Owner

ivmartel commented Apr 12, 2017

See #323 for more information on your Tool.info.position problem.

@vmaindarkar, could you explain what you mean when you say "I put the paging for images with 25 images"?

@ivmartel
Copy link
Owner

I like @Zaid-Safadi's solution too, I created #342 for that.

Closing this issue since it is a bit old, checkout #342 for progress!

@ivmartel ivmartel added duplicate This issue or pull request already exists wontfix This will not be worked on labels May 17, 2017
@ivmartel
Copy link
Owner

ivmartel commented May 17, 2017

Not sure about labels, wontfix now but will in, kind of duplicate, #342...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

5 participants