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

Support excel files in tabular module #201

Closed
blcham opened this issue Aug 24, 2023 · 3 comments · Fixed by #203
Closed

Support excel files in tabular module #201

blcham opened this issue Aug 24, 2023 · 3 comments · Fixed by #203
Assignees

Comments

@blcham
Copy link
Contributor

blcham commented Aug 24, 2023

Notes:

A/C:

  • within output RDF it is possible to query which columns or rows where merged
  • there is optional parameter to process only one sheet
  • functionality is tested with JUNIT tests including test with multiple sheets
  • module should support parsing multiple CSV files as well
  • if implemented using property holding mime-type value, it should be checked that combination of delimiter value and mime-type values are together consistent
  • TabularModule main documentation is revised by @blcham (especially default behaviour, process-specific-sheet-in-xls-file should be refactored to address multiple tables, is it ok to replace cell value '\t' to ' ' in HTML and excel? )

Should be implemented after #204

@blcham
Copy link
Contributor Author

blcham commented Aug 24, 2023

FYI, regarding merged columns and rows -- this is how export to CSV works in google drive:
image

@blcham
Copy link
Contributor Author

blcham commented Aug 24, 2023

@rodionnv rodionnv self-assigned this Aug 24, 2023
@blcham
Copy link
Contributor Author

blcham commented Sep 2, 2023

@rodionnv I have added requirment to "module should support parsing multiple CSV files as well" (it is basically same as support for multiple excel sheets)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

2 participants