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

Stage ONT data for delivery #447

Merged
merged 10 commits into from
Dec 4, 2024

Conversation

ssjunnebo
Copy link

@ssjunnebo ssjunnebo commented Nov 28, 2024

  • Given a PID, look for project dirs in DATA, ANALYSIS and ONT_TAR
  • Make a staging directory “PID/TIMESTAMP” in DELIVERY
  • Symlink all the data to the staging dir

@ssjunnebo ssjunnebo added the WIP label Nov 28, 2024
@codecov-commenter
Copy link

codecov-commenter commented Nov 28, 2024

Codecov Report

Attention: Patch coverage is 0% with 109 lines in your changes missing coverage. Please review.

Project coverage is 27.95%. Comparing base (6960f95) to head (6277412).
Report is 16 commits behind head on master.

Files with missing lines Patch % Lines
taca/delivery/delivery_classes.py 0.00% 58 Missing ⚠️
taca/delivery/deliver.py 0.00% 28 Missing ⚠️
taca/delivery/cli.py 0.00% 23 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #447      +/-   ##
==========================================
- Coverage   28.17%   27.95%   -0.22%     
==========================================
  Files          34       37       +3     
  Lines        5068     5204     +136     
==========================================
+ Hits         1428     1455      +27     
- Misses       3640     3749     +109     

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

@ssjunnebo ssjunnebo requested review from alneberg and aanil December 3, 2024 09:45
@ssjunnebo ssjunnebo changed the title Stage data for delivery Stage ONT data for delivery Dec 3, 2024
taca/delivery/cli.py Outdated Show resolved Hide resolved
Copy link
Member

@aanil aanil left a comment

Choose a reason for hiding this comment

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

👍

@ssjunnebo ssjunnebo merged commit 661726f into NationalGenomicsInfrastructure:master Dec 4, 2024
6 of 7 checks passed
@ssjunnebo ssjunnebo deleted the stage_ont branch December 4, 2024 12:33
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.

3 participants