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.

Sign up for GitHub

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

Jump to bottom

Avoid global sort by removing indexInMatrix #32

Open
jakipatryk opened this issue Feb 11, 2023 · 1 comment
Open

Avoid global sort by removing indexInMatrix #32

jakipatryk opened this issue Feb 11, 2023 · 1 comment
Assignees
@jakipatryk
Labels
enhancement New feature or request wontfix This will not be worked on

Comments

@jakipatryk
Copy link
Owner

Subtask of #11.

Global sort is an expensive operation, that doesn't seem to be unavoidable. Try to remove indexInMatrix and all dependencies of it by utylizing threshold instead to bucket the matrix on horizontal axis, and threshold+dim to sort within partition.
@jakipatryk jakipatryk added the enhancement New feature or request label Feb 11, 2023
@jakipatryk jakipatryk added this to the 0.1.0 release milestone Feb 11, 2023
@jakipatryk jakipatryk self-assigned this Feb 11, 2023
@jakipatryk jakipatryk added the wontfix This will not be worked on label Mar 11, 2023
@jakipatryk
Copy link
Owner Author

Doesn't seem to be (easly) doable. Removing from 0.1.0 release.

@jakipatryk jakipatryk removed this from the 0.1.0 release milestone Mar 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jakipatryk jakipatryk

Labels
enhancement New feature or request wontfix This will not be worked on
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jakipatryk