Skip to content
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

ENH: improve DataFrame read_csv / to_csv for Index/MultiIndex #151

Closed
wants to merge 1 commit into from

Conversation

jseabold
Copy link
Contributor

2 issues for DataFrame.to_csv

1 ) If header and index are True, you should write something for an index header, even if index_level is None.
2) If given a MultiIndex, don't want to just call str() on each element.

Also fixed read_csv to take an iterable for index_col so that I could write tests.

I haven't looked at the similar methods in Series or the other to_* methods yet. This is all I needed at the moment.

Aside: it looks like in test_frame setUp isn't a classmethod and is called for each test method. Not a big deal, but might be a little heavy in the test suite.

@wesm
Copy link
Member

wesm commented Sep 22, 2011

thanks a lot for doing this. after hemming/hawing i think it makes sense to have a default index header column. rebased into master

I think it would be a good idea to make it so a regular Index can have a name attribute. Will create a separate github issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants