-
Notifications
You must be signed in to change notification settings - Fork 7
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
Exception of type 'System.Exception' was thrown #17
Comments
Should be more reliable now, I'll keep this issue open for a bit to see if it comes up again. |
I can get these pretty reliably in Grubs.
Reproduction steps:
I find I can get the error pretty regularly this way, but it is not consistent. And often, after experiencing the error once, it does not happen again until a Grubs gamemode restart. |
Having a look again now. |
sbox-dev_TU2mwnTwLO.mp4Got a repro |
@trundlr I've pushed a new version that should be slightly improved, but the main thing is it'll give you a log like this instead of the old exception:
Could you get people to post them here whenever they happen, it'll help me find all the edge cases. Thanks! |
Sounds great! I'll see if I can just automate collecting all logs of this type after the contest, but for now, I'll post them when I see them. Thanks so much! |
After updating Grubs with the newest version, I seem to be getting these errors very consistently - basically every single startup. |
That's a really nice test case, lots going wrong there! sbox-dev_HZexJW5ltv.mp4Sorting this out is my priority now, sorry for the delay. |
No worries at all - this library is the only reason Grubs has been possible and the whole team appreciates everything you do ❤️ |
Current progress: fixed some edge cases in PolygonMeshBuilder, Edit: nevermind, red herring. More edges cases in PolygonMeshBuilder to solve. |
Hackweeking this week, so mostly won't be working on this. |
I got a bunch more with EdgeWidth of 0.5 and EdgeFaces of 3 |
FWIW I get these literally every single time in Grubs, just haven't been reporting actively. Not sure how much you want me to share.
|
Thanks for these, and sorry for the slow progress here. I'm pretty sure I'll need to do a more substantial rewrite rather than tweaking epsilons, since fixing one case tends to cause another to regress. Something I'm unsure about is if I can get away with using fixed point internally to avoid precision issues. |
That makes sense. I am curious why it started behaving so much worse after scene though 👀 We rarely received these issues in entity-system Grubs. Even our pre-existing texture levels get black squares and errors, when they never did before. |
The text was updated successfully, but these errors were encountered: