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

vDSP_vsortD possible array size too small #2

Open
ptoffy opened this issue Jan 16, 2025 · 0 comments
Open

vDSP_vsortD possible array size too small #2

ptoffy opened this issue Jan 16, 2025 · 0 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@ptoffy
Copy link
Member

ptoffy commented Jan 16, 2025

Currently vDSP_vsortD, which is supposed to sort a vector, can only sort it if it's size n is 0 < n < Int64.max, instead of the API's required 0 < n < UInt64.max due to not being able to use UInt as pointer index. We need to work around this.

Also, we're currently using LAPACK.dlasrt_ to sort it if the size n is 0 < n < Int32.max, and uses a custom QuickSort implementation for Int32.max <= n < Int64.max. Once #1 is solved we should switch to using this as default for performance reasons

@ptoffy ptoffy added bug Something isn't working enhancement New feature or request labels Jan 16, 2025
@ptoffy ptoffy changed the title vDSP_vsortD issue vDSP_vsortD possible array size too small Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant