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

ArrSorting sending more than one chunk to repaint() in publish() #23

Closed
dlarocque opened this issue Aug 18, 2020 · 0 comments
Closed

ArrSorting sending more than one chunk to repaint() in publish() #23

dlarocque opened this issue Aug 18, 2020 · 0 comments
Labels
bug Something isn't working

Comments

@dlarocque
Copy link
Owner

Since some systems are able to run the program faster than others, the program will send more than one chunk to publish(List<Integer[]> chunk), and causing the program to fall behind in frames drawn, making the red and blue bars that draw the swapped indexes out of sync with the sorting visualization.

Simply putting a very long sleep time makes the larger sorting arrays very slow to sort, and doesn't entirely fix the issue.

Need to find a way to either make the Sleep() method sleep an amount of time based on the system, or make the publish() method able to draw all the chunks to it, no matter how many there are.
Fixing this could open up the possibility of adding a slider for changing sorting speed.

@dlarocque dlarocque added the bug Something isn't working label Aug 18, 2020
dlarocque added a commit that referenced this issue Sep 4, 2020
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
Development

No branches or pull requests

1 participant