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

Memory issues with TMTV mode #2968

Closed
BrandonDatUHN opened this issue Sep 30, 2022 · 4 comments
Closed

Memory issues with TMTV mode #2968

BrandonDatUHN opened this issue Sep 30, 2022 · 4 comments

Comments

@BrandonDatUHN
Copy link

I posted this video before https://watch.screencastify.com/v/poYbBEarUdmzvUhty0J3 which shows what is happening but since it didn't seem to happen on each machine we tested I've done some deeper digging.

We made a google doc to log our tests and I'm hoping the error in the console might point you in the right direction. of the 4 computers we tested on 2 had the error and 2 did not. Interestingly the two that had it do not have GPU and the two that did not do. All had 16 GB RAM.

https://docs.google.com/document/d/1hVltQfBsK9cK-jvB4aBjeAvHIkxF3uDl1Ij8cKk-mIo/edit?usp=sharing

@sedghi
Copy link
Member

sedghi commented Oct 7, 2022

this is a great documented step by step report. Thanks for taking time in filling it.

For each setup can you put a screenshot of the following page? https://webglreport.com/?v=2

@BrandonDatUHN
Copy link
Author

Posting some more information here about this bug after using the chrome task manager. It appears that the memory that is causing the crash is from GPU Process. (This is on my laptop with integrated graphics)

Some advice from the web suggests that if you turn off "use hardware acceleration when available " in chrome the problem will go away and that seems to be correct.

This is a screenshot loading WITH hardware acceleration enabled
image

and here is one without

image

Here is the webglreport

image

Additionally, you asked for the memory dumps/screenshots before and after opening a study. Here they are along with screen shots (this is with hardware acceleration off.

Before loading:
image

After loading in viewer

image

Back at the study list
image

The memory snapshots can be found here

https://fileshare.uhn.ca/download/82413d55-c71a-4b2c-b6fe-984a558ada84
Please note that this file upload will expire in 30 days.

@BrandonDatUHN
Copy link
Author

This issue still exists on version 3.3 however I can't reproduce on the demo anymore because I get this error instead.

image

on our release we set the number of workers to 1 which removes that error but the one above still exists,

@sedghi
Copy link
Member

sedghi commented Oct 13, 2023

#3207

@sedghi sedghi closed this as completed Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants