Skip to content

feat: csvs into xlsx - #143

Draft
g-pechorin wants to merge 5 commits into
mainfrom
feat.csvs-into-xlsx
Draft

g-pechorin wants to merge 5 commits into
mainfrom
feat.csvs-into-xlsx

Conversation

@g-pechorin

Copy link
Copy Markdown
Contributor
✨ Feature

PR Description

Adds functions to combine/split .csv files into .xlsx files.

Intention is to build a pipeline that;

  • runs carrot-transform (et al) on the extracted data
  • merges those .csv (or .tsv?) it into a .xlsx document
  • run the .xlsx through existing shift_dates_inplace()
  • split the .xlsx file into .csv / .tsv again
  • upload those to the BC Insights

Related Issues or other material

Related #https://github.com/Health-Informatics-UoN/nuh-sql/issues/89
Closes #

Screenshots, example outputs/behaviour etc.

✅ Added/updated tests?

  • This PR contains relevant tests / Or doesn't need to per the below explanation

[optional] What gif best describes this PR or how it makes you feel?

This feels surprisingly "clean" as an approach

alt_text

@github-actions

github-actions Bot commented Sep 23, 2026

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissBranchBrPartCoverMissing
nuh_helper/csv_xlsx
   __init__.py43216293%19, 64
nuh_helper/date_shift
   __init__.py278181061689%149->158, 167, 255->253, 257->251, 376->390, 400->415, 402, 552, 578, 601, 614, 622–629, 665, 687, 691, 697
   _excel.py942256771%27->26, 34, 124–137, 174, 207->213, 220–231
   _parse.py30516285%31–40
nuh_helper/profile
   __init__.py178254298%180->183, 255–256
TOTAL643492522989% 

Tests Skipped Failures Errors Time
97 0 💤 0 ❌ 0 🔥 8.980s ⏱️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants