Skip to content

Conversation

@jorisvandenbossche
Copy link
Member

closes #5435. Clarified that ExcelFile is not deprecated but just moved. I don't use Excel myself, so can somebody check if it makes sense what I wrote? (@

For now, I didn't add ExcelFile to api.rst (but ExcelFile.parse is already in there, so it is already in some way mentioned), as I am not sure if all it's methods (mro?) should be public (this has possibly to be cleaned up first)

@hayd
Copy link
Contributor

hayd commented Jan 27, 2014

I think this makes sense, atm i like to read in Excel (if I really have to!) as a dict with sheet names:

xl = ExcelFile('foo.xls')
{sheet: xl.parse(sheet) for sheet in xl.sheet_names}

Note sure there is a way to do this without ExcelFile.

@jreback
Copy link
Contributor

jreback commented Jan 28, 2014

@jorisvandenbossche looks good to me ...bombs away

jreback added a commit that referenced this pull request Jan 28, 2014
DOC: clarify that ExcelFile is not deprecated but only moved (#5435)
@jreback jreback merged commit e397215 into pandas-dev:master Jan 28, 2014
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.

Add ExcelFile to API docs + clarify ExcelFile isn't deprecated (just moved)

3 participants