-
Notifications
You must be signed in to change notification settings - Fork 5
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
Fast and accurate float to string conversion (ryu) #104
Comments
A talk explaining Ryū: https://www.youtube.com/watch?v=kw-U6smcLzk |
Wouldn't it be more ideal if the standard lib used ryu itself or is the possibility that that would break something too great? |
Don't you think a first implementation should be confined to a library so everyone can A/B it against stdlib and discover bugs, pain points, or regressions? Even an impure version would be a useful starting point. |
I mean, you're right, I was just wondering if the ultimate goal here was to upgrade the standard lib |
I mean, you're right; the goal would be fast and accurate float conversion by default. 😄 |
I looked into this a bit and nim appears to simply |
Thumbs up from me, a stdlib patch will be accepted. |
IIRC, the creator mentioned that the algo for Ryu is only like 100 lines, though I looked at the source and it seemed quite a bit more extensive. If no one else wants it, I'll try to contribute; t'd be a fun first project. I'd guess that we'd want to land somewhere between the state-of-the-art C implementation and a 100 line paper implementation in terms of complexity. 😉 |
we now have dragonbox (refs nim-lang/Nim#18139), we should close this ( I don't have permissions to close) |
I've long been enamored with this one.
https://github.com/ulfjack/ryu
The text was updated successfully, but these errors were encountered: