Skip to content

Don't check for pod5/fast5 files#492

Merged
ssjunnebo merged 2 commits intoNationalGenomicsInfrastructure:masterfrom
ssjunnebo:pod5
Oct 7, 2025
Merged

Don't check for pod5/fast5 files#492
ssjunnebo merged 2 commits intoNationalGenomicsInfrastructure:masterfrom
ssjunnebo:pod5

Conversation

@ssjunnebo
Copy link

Remove the assert for raw data

@codecov
Copy link

codecov bot commented Oct 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 26.04%. Comparing base (6e97e94) to head (2dfde26).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #492      +/-   ##
==========================================
- Coverage   26.05%   26.04%   -0.02%     
==========================================
  Files          37       37              
  Lines        5545     5544       -1     
==========================================
- Hits         1445     1444       -1     
  Misses       4100     4100              

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

@alneberg
Copy link
Member

alneberg commented Oct 7, 2025

Ah, yes. Anything else we've missed or do you want a review now?

@ssjunnebo
Copy link
Author

Review please 😄
I've tested it on the preproc and it seems to work OK

Copy link
Member

@alneberg alneberg left a comment

Choose a reason for hiding this comment

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

All good. Versions come quickly now.

@ssjunnebo ssjunnebo merged commit 7d9d24c into NationalGenomicsInfrastructure:master Oct 7, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants