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

(re)add Excel reader #31

Open
simleo opened this issue Jun 23, 2017 · 0 comments
Open

(re)add Excel reader #31

simleo opened this issue Jun 23, 2017 · 0 comments
Assignees

Comments

@simleo
Copy link
Member

simleo commented Jun 23, 2017

While refactoring the ICY reader (see #21) we removed a generic Excel to CSV converter, xls_to_csv. As noted in the PR comments, a generic Excel reader can be useful for other formats. We can either readd that function, perhaps in another module since it does not follow the same read-file-to-dataframes protocol as current readers, or develop a generic Excel reader based on it (writing to CSV would be accomplished by the data package writing code).

@simleo simleo self-assigned this Jun 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant