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

Enable loading Excel files as data tables #693

Open
brynpickering opened this issue Sep 30, 2024 · 2 comments
Open

Enable loading Excel files as data tables #693

brynpickering opened this issue Sep 30, 2024 · 2 comments
Labels
enhancement v0.7 (upcoming) version 0.7

Comments

@brynpickering
Copy link
Member

What can be improved?

For some projects, it can be useful to store data tables in one Excel file. We'd just need the extra sheets option available in our data table definition. Of course, the standard caveats of working with Excel apply (namely, the datetime format automatically changing on save to match the device format).

Version

v0.7.0.dev5

@brynpickering brynpickering added enhancement v0.7 (upcoming) version 0.7 labels Sep 30, 2024
@irm-codebase
Copy link
Contributor

@brynpickering is it possible to add this feature without making our tables functionality super complex? In all honesty I think we'd gain more from supporting better defined formats (like parquet) than all the issues that come from trying to support excel...

@brynpickering
Copy link
Member Author

We can add parquet to the list, there is even an issue for this to be added in v0.6 #694. In v0.7 it is possibly just a matter of updating the allowed suffixes with no change in the data table syntax. For excel it just requires a sheet_name option in the data table definition and fastexcel as an optional dependency, so I wouldn't say it will make it super complex.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement v0.7 (upcoming) version 0.7
Projects
None yet
Development

No branches or pull requests

2 participants