-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Crash on inf
value in CSV
#2400
Comments
Probably the same as #2349
|
Yeah, I also found that bug. Figured it was different enough (float vs int) to warrant another issue. Good news is that there's a PR in the |
I also created akubera/bigdecimal-rs#67, but if a PR has been sitting around this long, it may not end up getting addressed. If there's no movement, I think there are some other options:
|
Describe the bug
When the value
inf
is found in a CSV file, the BigDecimalFrom<f64>
implementation crashes. I haven't finished investigating, but it seems like that crate assumes allf64
s can be turned into a BigDecimal. Not sure they accounted forinf
values. I was going to open a PR when I found the issue, but now I''m not sure if this is better solved in the BigDecimal crate or innu_protocol::UntaggedValue::decimal
.To Reproduce
Steps to reproduce the behavior:
bad.csv
.nu
).open bad.csv
. This step should panic.Expected behavior
I expected it to display the CSV file. It crashes instead.
Screenshots
If applicable, add screenshots to help explain your problem.
The text was updated successfully, but these errors were encountered: