-
Notifications
You must be signed in to change notification settings - Fork 113
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
Idea for definition of conversion constants #81
Comments
Sorry for not getting back to you earlier. I like this idea a lot. I'm a little worried that we'd get collisions because someone else might define In practice, I'd imagine we could avoid this by using the irrational macro with very unique names like Also, while I defined I'll get to this eventually but would also welcome a PR if you're feeling up for it. |
Thank you! I like your approach to avoid type collision. I might have some time to bake a PR, but not anytime soon. |
With #754, floating-point conversion factors are now |
Currently, the package isn't very arbitrary-precision-friendly, because conversion constants are often of
Float64
type. My idea is to define them asIrrational
:The last three lines show that in this way the conversion factor has full precision as a
BigFloat
.Edit: I initially proposed to define a type that was a simplification of
Irrational
, but in the end there wasn't much difference between my custom type andIrrational
.The text was updated successfully, but these errors were encountered: