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

Make rayon optional #52

Open
bminixhofer opened this issue Mar 5, 2021 · 0 comments
Open

Make rayon optional #52

bminixhofer opened this issue Mar 5, 2021 · 0 comments
Labels
P3 Low Priority

Comments

@bminixhofer
Copy link
Owner

rayon is not necessary and should be behind a feature flag (like in e.g. ndarray). Resolving this issue in rayon-cond would make it trivial: cuviper/rayon-cond#3

If this change does not get accepted in rayon-cond I believe the best solution is forking it, adding a rayon feature flag and also moving the abstraction currently in src/utils/parallelism.rs to the fork.
@bminixhofer bminixhofer added the P3 Low Priority label Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P3 Low Priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bminixhofer