uab-cgds-worthey / quac

🦆 Quality Control of WGS and exome samples 🦆
https://quac.readthedocs.io
GNU General Public License v3.0
5 stars 1 forks source link

Make output files write-protected #9

Open ManavalanG opened 3 years ago

ManavalanG commented 3 years ago

In GitLab by @ManavalanG on Apr 6, 2021, 08:45

ManavalanG commented 3 years ago

mentioned in commit f7ddc28e72bc6ce9a6020e11e190de7c8d1d94fb

ManavalanG commented 3 years ago

Not all files tagged with protected are made read-only by snakemake. Another one of snakemake's quirks? :angry:

ManavalanG commented 3 years ago

Question: Is this a bug in snakemake v6.0.5, which is used in QuaC pipeline? Is it also present in snakemake v5.9.1?

Ran QuaC pipeline with snakemake v5.9.1. It behaved the same way as did v6.0.5. That is, the same selective files that were write protected by v6.0.5 are the only ones write protected by v5.9.1.

manag@login001:/scratch/manag/tmp/quac/results/test_project/analysis$ fd -l | grep -e "-r--r--r--"
-r--r--r--  1 manag manag  54K Apr 11 18:12 project_level_qc/covviz/covviz_report.html
-r--r--r--  1 manag manag 2.0K Apr 11 18:11 project_level_qc/indexcov/indexcov-indexcov.bed.gz
-r--r--r--  1 manag manag 7.1K Apr 11 18:11 project_level_qc/indexcov/index.html
-r--r--r--  1 manag manag 1.6K Apr 11 18:12 project_level_qc/mosdepth/mosdepth.html
-r--r--r--  1 manag manag  166 Apr 11 18:11 project_level_qc/multiqc/aggregated_rename_configs.tsv
-r--r--r--  1 manag manag 1.8M Apr 11 18:13 project_level_qc/multiqc/multiqc_report.html
-r--r--r--  1 manag manag 271K Apr 11 18:13 project_level_qc/somalier/ancestry/somalier.somalier-ancestry.html
-r--r--r--  1 manag manag 211K Apr 11 18:13 project_level_qc/somalier/ancestry/somalier.somalier-ancestry.tsv
-r--r--r--  1 manag manag  23K Apr 11 18:12 project_level_qc/somalier/relatedness/somalier.html
-r--r--r--  1 manag manag  211 Apr 11 18:12 project_level_qc/somalier/relatedness/somalier.pairs.tsv
-r--r--r--  1 manag manag  440 Apr 11 18:12 project_level_qc/somalier/relatedness/somalier.samples.tsv

Conclusion: Switching to v5.9.1 would not solve the problem.