-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
read csv thousands separator #4322
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
Comments
Possibly this is a dupe of #2594 |
@jreback not sure what you're asking? |
is this a bug? e.g. the dtypes not working with thousands or a bug in thousands sep? or a feature request (which is there already)? |
@jreback this is a bug, thousand separator doesn't seem to work in the docs you link to it says
in the docstring for read_csv it asks for:
Doesn't seem to be working with |
ahh ok...i c now..... |
Adds support for the thousands character in csv parser for floats. Updated docs to reflect bug fix.
closed by #4598 |
@jreback The date aspect of this pr is still not fixed. For some reason the thousands separator attacks the date and makes it a string. |
Wow, so this is a very edge case... it's cos the date column is just Not sure what solution is. |
but it should ignore dates columns entirely (for thousands parsing...).....hmmm...why don't you open a separate issue and can cross-link it |
From this SO question, with input to include thousands:
Note: the Dates column (as well as the thousands not being converted.
The text was updated successfully, but these errors were encountered: