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

Switch CPU sorting from home-spun bubble sort #32

Open
lorton opened this issue Aug 14, 2018 · 1 comment
Open

Switch CPU sorting from home-spun bubble sort #32

lorton opened this issue Aug 14, 2018 · 1 comment
Assignees

Comments

@lorton
Copy link
Member

lorton commented Aug 14, 2018

We are currently using a home-spun bubble sort that had guaranteed time (return_count * N, where return count is very small). Now that we're allowing users to define return count, and they could potentially define it large, this implementation is bad.

@lorton lorton self-assigned this Aug 14, 2018
@lorton
Copy link
Member Author

lorton commented Aug 14, 2018

This is in response to your discussion in the review comment @ricrogz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant