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

setBackground - mayo shoulder data crash #244

Closed
amymmorton opened this issue Dec 20, 2023 · 3 comments · Fixed by #245
Closed

setBackground - mayo shoulder data crash #244

amymmorton opened this issue Dec 20, 2023 · 3 comments · Fixed by #245
Assignees
Labels
bug Something isn't working

Comments

@amymmorton
Copy link
Collaborator

Tested from #76

The mayo radiograph test set (shldimg0005 Prop_Int) I have are incredibly 'dense' black when rendered. I wanted to see if setBackground could provide a better input to the ncc calc

on setBackground (following threshold popup) a QWindowsWindow error is returned in cmd and and unhandled exception thrown line 186 Video.cpp

setBackground_QWindow
shoulder_drr_radDARK

With the Knee data (311135_L_Hop) the same QWindowWindows error message is populated in cmd but the unhandled exception does not occur
setBackground_KNEEQWindow

@amymmorton amymmorton added the bug Something isn't working label Dec 20, 2023
@NicerNewerCar
Copy link
Contributor

NicerNewerCar commented Dec 20, 2023

@amymmorton
I was able to get a different error, by loading the same data from #203. I am getting errors within both the CUDA and OpenCL backends, the same values I tested work on the sample data.

I have not been able to reproduce the error within the Video.cpp yet. Could you send me this dataset?

In order to set the background you need at minimum 2 images per camera. This should definitely fail more gracefully than some random CUDA/OpenCL errors...

Investigating the pointer error now.

The QWindowsWindow error is something that has been happening for a while during the start-up of the UI and will be addressed when the larger UI improvement starts.

@amymmorton
Copy link
Collaborator Author

I create a truncated dataset for debugging (8 frames)

@amymmorton
Copy link
Collaborator Author

And agreed- I had the error on both cuda/opencl backend builds

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
2 participants