Skip to content

New xsd/pyarrow XML data loader - #293

Merged
ialarmedalien merged 4 commits into
developfrom
xmltopyarrow
Oct 21, 2026
Merged

ialarmedalien merged 4 commits into
developfrom
xmltopyarrow

Conversation

@ialarmedalien

Copy link
Copy Markdown
Collaborator

No description provided.

@codecov

codecov Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.76510% with 61 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.78%. Comparing base (86a0d1b) to head (80a500e).
⚠️ Report is 1 commits behind head on develop.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/cdm_data_loaders/readers/xsd.py 93.29% 34 Missing ⚠️
...dm_data_loaders/pipelines/xmltopyarrow/pipeline.py 0.00% 22 Missing ⚠️
...dm_data_loaders/pipelines/xmltopyarrow/settings.py 90.90% 3 Missing ⚠️
...c/cdm_data_loaders/pipelines/xmltodict/pipeline.py 0.00% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #293      +/-   ##
===========================================
+ Coverage    80.01%   80.78%   +0.76%     
===========================================
  Files           90       92       +2     
  Lines         6565     7093     +528     
===========================================
+ Hits          5253     5730     +477     
- Misses        1312     1363      +51     
Files with missing lines Coverage Δ
src/cdm_data_loaders/pipelines/ncbi_ftp_promote.py 0.00% <ø> (ø)
...c/cdm_data_loaders/pipelines/xmltodict/settings.py 100.00% <100.00%> (+8.10%) ⬆️
...rs/readers/jsonschema_xsv/xsv_validator/helpers.py 100.00% <ø> (ø)
src/cdm_data_loaders/readers/xml.py 96.26% <100.00%> (+1.26%) ⬆️
...c/cdm_data_loaders/pipelines/xmltodict/pipeline.py 0.00% <0.00%> (ø)
...dm_data_loaders/pipelines/xmltopyarrow/settings.py 90.90% <90.90%> (ø)
...dm_data_loaders/pipelines/xmltopyarrow/pipeline.py 0.00% <0.00%> (ø)
src/cdm_data_loaders/readers/xsd.py 93.17% <93.29%> (-1.11%) ⬇️

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 43ba45f...80a500e. Read the comment docs.

@ialarmedalien
ialarmedalien merged commit 5977261 into develop Oct 21, 2026
19 of 21 checks passed
@ialarmedalien
ialarmedalien deleted the xmltopyarrow branch October 21, 2026 20:43
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.

1 participant