sheet_utils refactor to add csv functionality (C4-1088) - Githubissues

4dn-dcic / utils

various util modules shared amongst several projects in our organization

MIT License

4 stars 1 forks source link

sheet_utils refactor to add csv functionality (C4-1088) #276

Open netsettler opened 1 year ago

netsettler commented 1 year ago

This is an improvement on Some tools for working with spreadsheets (PR #275) to generalize it a bit more and add support for .csv files.

From the CHANGELOG.rst...

New module sheet_utils for loading workbooks.
- Important things of interest:
- Class ItemManager for loading Item-style data from either .xlsx or .csv files.
- Function load_items that does the same as ItemManager.load.
- Various low-level implementation classes such as:
- Classes XlsxManager and CsvManager for loading raw data from .xlsx and .csv files, respectively.
- Classes ItemXlsxManager and ItemCsvManager for loading Item-style data from .xlsx and .csv files, respectively.
Contains a fix for a bug in ff_utils.get_schema_names (C4-1086).

netsettler commented 1 year ago

I like the organization here, and the consideration for csv files which can easily be adapted for tsv files.

Yeah, I can probably trivially add tsv files. I'll look at that.