-
Notifications
You must be signed in to change notification settings - Fork 48
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 in thread "ComparisonSorting", index 32768 out of bounds for length 32768 #110
Comments
Please provide a clear step-by-step procedure of how to reproduce this, along with all files and code modifications you made. Otherwise, (1) we have literally no way to debug this, and (2) it could be that the issue is with the code you made. |
Step by step instruction to reproduce:
2024-07-13.11-14-43.mp4It seems to happen in the clearAllMarks function. SO maybe a highlight was placed at 32768? Or maybe the length 32768 is so long it bugs out sorts (like FlammerSort), but if a highlight was placed at 32768 (out of bounds) the Highlights.markArray(1, 32768); would've thrown an out of bounds error at me but it didn't. it was the clear all marks function. |
I'm confused, isn't there clearly an out-of-bounds error displayed in an error popup before you check the console? Anyway, the error occurs while the sort is still running, so it seems like it's an issue with your own sort. |
What could the issue be then? Also, before i did the recording, there was no error message that popped up. |
I've traced down the reason, described in this PR #101 |
When a sort ended on length 32768 length, the visual freezes, highlights and sound are frozen, but not cause of a freeze bug, it was beacuse it seemed when it tried to end the sort, the clear all marks bugged out.
I dont know why, but i didnt edit the highlight code, so it must be the same from my to your fork. It happens when i disable "End sweep anim", and when i am using a custom sort "Adaptive Sort" i made but not when i am using any other sorts. I do not get why it does not work, as if it was a mark index that was out of bounds (example Highlights.markArray(1, 32768) at 32768 length) it would've thrown an error at me, but it didn't. Yes, i tested importing my sort into your fork (with the added importation of MathExtra from the code) but it did not work.
The text was updated successfully, but these errors were encountered: