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.

Sign up for GitHub

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

Jump to bottom

Read from IO #26

Open
felipenoris opened this issue May 15, 2018 · 2 comments
Open

Read from IO #26

felipenoris opened this issue May 15, 2018 · 2 comments
Labels
enhancement New feature or request

Comments

@felipenoris
Copy link
Owner

felipenoris commented May 15, 2018 •
edited

Use case: Excel file inside a Zip or Gzip file.
@felipenoris felipenoris added the enhancement New feature or request label May 15, 2018
@felipenoris felipenoris mentioned this issue Feb 28, 2019
Closed
felipenoris added a commit that referenced this issue Feb 28, 2019
@felipenoris
read XLSX from stream (#26)
5d925c1
@Arkoniak Arkoniak mentioned this issue May 5, 2020
Open
@Arkoniak
Copy link

Arkoniak commented May 5, 2020

It would be nice to read xlsx directly from the web, without intermediate storage on a hard drive. Looking at the source it seems like it's doable:

  1. XLSXFile should have an additional constructor (which accepts io instead of filepath)
  2. read_or_open_xlsx should split itself based on the way how data is ingested.

Is it correct? Or something else is needed?

@felipenoris
Copy link
Owner Author

felipenoris commented May 5, 2020 •
edited

@Arkoniak, yes that is basically it. Feel free to send a PR on this.
Maybe this will trigger a major refactoring.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Arkoniak @felipenoris