-
Notifications
You must be signed in to change notification settings - Fork 140
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
Failure to read file with big integers (Int128) #769
Comments
quinnj
added a commit
to JuliaData/Parsers.jl
that referenced
this issue
Nov 10, 2020
Fixes JuliaData/CSV.jl#769. The issue here is our `digits` variable for tracking all digits we've parsed in decimal form is an unsigned integer, but might be converted to a signed integer in cases where there's no decimal point or a decimal point with no trailing digits. In those cases, there's an edge case of overflow because the overflow check was on the _unsigned_ integer instead of the _signed_ integer, which led to the conversion error. The fix is to instead change the overflow check to be on the _signed_ integer.
Thanks for the report @DrChainsaw! A fix is up: JuliaData/Parsers.jl#71 |
quinnj
added a commit
to JuliaData/Parsers.jl
that referenced
this issue
Nov 10, 2020
* Fix signed integer overflow case for float parsing Fixes JuliaData/CSV.jl#769. The issue here is our `digits` variable for tracking all digits we've parsed in decimal form is an unsigned integer, but might be converted to a signed integer in cases where there's no decimal point or a decimal point with no trailing digits. In those cases, there's an edge case of overflow because the overflow check was on the _unsigned_ integer instead of the _signed_ integer, which led to the conversion error. The fix is to instead change the overflow check to be on the _signed_ integer. * simplify
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This seems to depend on the number of columns and rows. Is it some type-fitting heuristic which fails?
The text was updated successfully, but these errors were encountered: