-
Notifications
You must be signed in to change notification settings - Fork 809
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
Array.Parallel.Sort* family of functions #14820
Array.Parallel.Sort* family of functions #14820
Conversation
…llection-functions-for-arrayparallel-more-regular
Same question here: Wondering if the APIs should be marked as |
As per #14827 (comment) , I am now more inclined to put the attribute in place here. |
…rayparallel-more-regular
…llection-functions-for-arrayparallel-more-regular
…version:preview'")>]
…llection-functions-for-arrayparallel-more-regular
…llection-functions-for-arrayparallel-more-regular
…rayparallel-more-regular
tests/FSharp.Core.UnitTests/FSharp.Core/Microsoft.FSharp.Collections/ArrayModule2.fs
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, we can probably get some DevDev algorithm gurus to look at all this code.
@vzarytovskii maybe you know someone who can help? Or should we ask @stephentoub right away? :)
tests/FSharp.Core.UnitTests/FSharp.Core/Microsoft.FSharp.Collections/ArrayModule2.fs
Outdated
Show resolved
Hide resolved
…ctions/ArrayModule2.fs Co-authored-by: Petr <[email protected]>
…rayparallel-more-regular
…rayparallel-more-regular
…rayparallel-more-regular
…rayparallel-more-regular
* Parallel sorting impl
This is first part of the RFC to expand the Array.Parallel module, based on approved suggestion
Benchmarks were used for picking an appropriate implementation, comparing againts:
Sequential Array.sort
PLINQ with varying degree of parallelism
Surface area being extended by the following functions.
Main 3 categories: