Skip to content
This repository has been archived by the owner on Dec 11, 2024. It is now read-only.

remove rayon from hashbrown dep #29

Closed
wants to merge 1 commit into from
Closed

Conversation

lthiery
Copy link

@lthiery lthiery commented Jan 13, 2022

When trying to use h3ron v0.12.0 without any features, rayon still gets pulled in. This is because hasbrown now depends on it as a feature, but it appears it might be unnecessary?

@lthiery
Copy link
Author

lthiery commented Jan 13, 2022

Nevermind - looking more closely it does look like you lose the parallel iterator without this flag. It might be cool to do something like this to make rayon an option for h3ron clients. But I'll try that another time perhaps.

@lthiery lthiery closed this Jan 13, 2022
@nmandery
Copy link
Owner

Thanks for bringing this up - I had not noticed this. The activated hashbrown features where generally in conflict with the h3ron featuregates. 65abf80 should fix this.

@lthiery
Copy link
Author

lthiery commented Feb 24, 2022

Thanks for bringing this up - I had not noticed this. The activated hashbrown features where generally in conflict with the h3ron featuregates. 65abf80 should fix this.

Thanks for fixing this up!

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

Successfully merging this pull request may close these issues.

2 participants