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

Link project to sample #1

Merged
merged 5 commits into from
May 10, 2022
Merged

Link project to sample #1

merged 5 commits into from
May 10, 2022

Conversation

CuijieLu
Copy link
Contributor

@CuijieLu CuijieLu commented May 3, 2022

No description provided.

@CuijieLu CuijieLu requested a review from dmcmanam May 3, 2022 21:18
Copy link
Member

@dmcmanam dmcmanam left a comment

Choose a reason for hiding this comment

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

What is the current test coverage percentage?

# step 1 get project ID as input, query from db to get fastq list eg :http://delphi.mskcc.org:8080/ngs-stats/permissions/getRequestPermissions/13117_B
# step 2 create symbol links eg: ln -sf /igo/delivery/FASTQ/RUTH_0089_AHHLYJDSX3/Project_13117_B/Sample_HCTWT1_IGO_13117_B_1 /igo/delivery/share/bakhoums/Project_13117_B/RUTH_0089
# step 3 call setaccess
def main():
Copy link
Member

Choose a reason for hiding this comment

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

Can we change the function signature to take the project as an argument? Then it will be ready to call from the crontab.

@dmcmanam dmcmanam merged commit 155d4d0 into main May 10, 2022
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