Skip to content

Add thrombosis AF project - #678

Merged
stefpiatek merged 19 commits into
mainfrom
stefpiatek/add_thrombosis_af
Sep 21, 2026
Merged

stefpiatek merged 19 commits into
mainfrom
stefpiatek/add_thrombosis_af

Conversation

@stefpiatek

Copy link
Copy Markdown
Contributor
  • Tweak logging levels
  • Update documentation
  • Keep scan options by default as we don't expect PID in this

@codecov

codecov Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 88.41%. Comparing base (5dd5114) to head (6635679).

Files with missing lines Patch % Lines
pixl_dcmd/src/pixl_dcmd/dicom_helpers.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #678   +/-   ##
=======================================
  Coverage   88.41%   88.41%           
=======================================
  Files          81       81           
  Lines        3972     3972           
=======================================
  Hits         3512     3512           
  Misses        460      460           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@p-j-smith p-j-smith left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah the logging changes are a good shout

Comment thread projects/configs/README.md
@stefpiatek
stefpiatek merged commit e69648d into main Sep 21, 2026
10 of 11 checks passed
@stefpiatek
stefpiatek deleted the stefpiatek/add_thrombosis_af branch September 21, 2026 13:45
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