Make `rayon` dependency optional?

sarah-ek / faer-rs

Linear algebra foundation for the Rust programming language

https://faer-rs.github.io

MIT License

1.79k stars 58 forks source link

Make `rayon` dependency optional? #71

Closed CGMossa closed 10 months ago

CGMossa commented 11 months ago

Is it possible to make rayon-dependency optional? If so, would you consider accepting a PR that does this?

sarah-ek commented 10 months ago

is there a reason for this?

CGMossa commented 10 months ago

None in particular, just

Fewer dependencies initially when working with faer,
Perhaps an expectation since everyone does it?
For small enough matrices, I'm not sure rayon is even important anyways, even though this is supposed to be used with big-stuff.

sarah-ek commented 10 months ago

fair enough. i'll make it optional (enabled by default) then

CGMossa commented 10 months ago

Thanks! Note, you don't have to do everything. I thought I'd ask first. Then maybe attempt to do it. But of course it would be "easiest" if you did it.

sarah-ek commented 10 months ago

merged in #73