Changed uin8 to uint8 in response to issue #9266 #9350
Conversation
|
Thanks, this needs:
|
jreback
added
IO CSV
|
This is my first PR attempt and I don't understand this error message. Did I do something wrong in the test function I added? |
pandas/io/tests/test_parsers.py
Outdated
There was a problem hiding this comment.
I think the issue is that this line starts with extra whitespace, throwing off colspecs below.
|
@CoolRanch29 I rebase and push your branch up here. pls review and update this PR when you can I am not sure what you are trying to compare against here (e.g. what should the expected frame look like?) |
|
Okay, I added an expected frame to test again. The problem before is that pd.read_fwf wasn't working at all - an exception was thrown when trying to read the test case. I'm still concerned that in the resulting frame, the column 'dst' is of type 'object' when using read_fwf and of type 'float64' when created by DataFrame(). |
doc/source/whatsnew/v0.16.0.txt
Outdated
There was a problem hiding this comment.
needs fixing here (eliminate the merge lines)
|
whats new is now available for 0.16.1 5ebf521 |
|
can you rebase this? |
|
I'm pretty sure I just rebased. I'm sorry if this is causing a headache for you, I'm sure I'm doing some things stupidly. I've gotten a lot of benefit out of using pandas and so I'm trying to contribute but this is my first time contributing so I'm having to learn a lot about both the pandas code and github. |
pandas/io/tests/test_parsers.py
Outdated
There was a problem hiding this comment.
use
with tm.ensure_clean('test.txt') as path:
with open(path,'w') as f:
....
|
couple of style things. pls add a release note in v0.16.1 as well. |
The docstring listed 'method' before 'value' which is not consistent with the order of the arguments when calling the method.
Updated docs throughout DataFrame methods to mention that axis can be set to 'index' or 'column' instead of 0 or 1 which improves readability significantly.
closes #9266