-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DISP: show column dtype in DataFrame.info() output #5682
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
Conversation
👍 |
@cpcloud can you put an example of old and new up in the descr... |
sure thing |
I think their is an issue about adding the info method to Series, so maybe make a quick test method that assets that it raises (and ref this issue) - because series doesn't have a dtypes method (and this needs a modification for this change) |
ok |
0.14 starts next week, honor the bug-fix only period? |
@y-p ahh yes... sorry. merge after 0.13 rls |
It's clear nothing could possibly go wrong with something as simple as this. We've been there before... :) |
not sure what u mean |
what I mean is that the implementation would break for an info method on series (which doesn't exists at the moment) so I think that you should have a test of this method that assert raises with a series for attribute error |
Yep ok that's what I thought. Test is there. On Sunday, December 15, 2013, jreback wrote:
Best, |
i think this is fine |
DISP: show column dtype in DataFrame.info() output
Thanks @cpcloud. You used to be a code fountain not that long ago, did RL catch up with you? :) will add release notes. |
Got a job where I don't use Python. Had to put my pandas love on the back burner while I refamiliarize myself with C++ and learn a bunch of windows OS material. Not a whole lot of time on Linux. It brings me to tears occasionally. I'll try to at least bring it to a trickle in the coming months. |
please. trickle. :) |
if you're in microsoft land, LINQ and Rx are quite beautiful... |
I can add those release notes tomorrow if you don't beat me to it. Otherwise, much appreciated. |
Done 10 minutes ago, don't worry about it. |
closes #3429.
Old:
New: