-
Notifications
You must be signed in to change notification settings - Fork 2
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
AppCrash #125
Comments
The crash is most likly due to the Text Selection bug - that also solves #124 |
I still get this error for 3D Cubic, even with normalization The error message is almost identical but now also contains info about a runtime warning: https://imgur.com/j7olFzu |
Can you provide more info about the dataset and settings? Does it happen each time? Runtime warnings are due to dividing by zero or very small number close to zero - these produce np.Nan values which allow later for not drawing an empty object |
I checked different datasets and found an interesting thing: this error occurs only when loading full Examples->0520nm directory. If I load a single file or loading the Examples->0200nm directory, 3D Cubic widget works fine. Maybe my laptop really lacks available memory? Settings: default 3D Cubic settings, I do not touch anything Yes, for 0520nm directory it seems to happen every time. |
Does it happen for Arrow structures? |
I tried to check for Arrow, but got bug #133 instead. I also think it's memory, but can we provide a wrapper around it? I.e., instead of crashing the app would refuse to load a dataset that is too large and display warning? Or is it technically too difficult? |
(Same goes for #133 .)
Anyway, how about when larger averaging or decimation is selected? Weirdly enough, it seems like a purely Windows issue. I have noticed that somehow processing takes much longer here. |
Not 100% sure what caused it, maybe choosing not to normalize 3D vectors in a widget
The error message is attached as another image to: https://imgur.com/a/kHeqYxi
Edit: ok I think it was for 3D_CUBIC window actually, which could explain lack of memory error
The text was updated successfully, but these errors were encountered: