-
-
Notifications
You must be signed in to change notification settings - Fork 81
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
Tracking: num-traits
support
#157
Comments
Yeah absolutely. Can just make it an optional dependency and put the implementation under that feature flag :) Only reason it hasn't been done already is I haven't needed it and nobody else has implemented it :P |
Okidoki. I'll look into it soon. |
Would you like the trait |
I think separate file is probably best |
Shall we keep this open to track further |
Sure :) |
A user of my
uniform-cubic-splines
crate wanted to use it withultraviolet
and ran into a traits issue.My crate relies on
Zero
/One
traits. Those were removed fromstd::num
(unfortunately, I think), in 2014. My crate usesnum-traits::Zero
/One
.That means if some third party crate that implements a type that someone wants to use with my crate does not implement the
num-traits
version of these traits, it won't work. A user can't add support for it on their side (and neither can even my crate) because of orphan rules.I guess the only solution would be to add support for this to
ultraviolet
. FYI,cgmath
,glam
&nalgebra
all support these.Would you consider this at all (or a PR)?
The text was updated successfully, but these errors were encountered: