From ec15d0f9fdee90e6b751a402cb5150306a2c4307 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?S=C3=A9bastien=20Guizard?= Date: Thu, 11 Nov 2021 14:19:26 +0000 Subject: [PATCH] Update module: `pbccs` (#1018) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * 📦 NEW: First commit of pbccs module * 👌 IMPROVE: Remove option from command + rename output (ccs -> bam) * 👌 IMPROVE: Move .pbi output into report channel * 🐛FIX: Correct code after --rq option removal from command line module - module main.nf: Remove ramaining rq input channel - Test main.nf: Transfert rq into addParams - Test test.yml: Update md5sums * 🐛FIX: Repair additionnal option usage * 👌 IMPROVE: Add some pacbio test files * 🐛 FIX: Add Pacbio index to test_data.config * 👌 IMPROVE: CCS is run in parallel with --chunk option * 👌 IMPROVE: Add Pbindex in bam ouput channel * 👌 IMPROVE: Change label to process_low * 👌 IMPROVE: Define reports files names + add json version of txt report * 🐛 FIX: Add missing backslashes * 🐛 FIX: Add missing gz extension * 🐛 FIX: update ouput channel * 🐛 FIX: output file name * 👌 IMPROVE: .gitignore * 👌 IMPROVE: Update function.nf to last version * 👌 IMPROVE: Update saveAs in main.nf * 👌 IMPROVE: Add pbccs module * 🐛 FIX: Fix Broken test * 👌 IMPROVE: Update test_data.config * 🐛 FIX: Fix test * 👌 IMPROVE: Update path of test dataset files * 👌 IMPROVE: Remove useless index + Fix Typos * 📦 NEW: First commit of pbccs module * 👌 IMPROVE: Remove option from command + rename output (ccs -> bam) * 👌 IMPROVE: Move .pbi output into report channel * 🐛FIX: Correct code after --rq option removal from command line module - module main.nf: Remove ramaining rq input channel - Test main.nf: Transfert rq into addParams - Test test.yml: Update md5sums * 🐛FIX: Repair additionnal option usage * 👌 IMPROVE: Add some pacbio test files * 🐛 FIX: Add Pacbio index to test_data.config * 👌 IMPROVE: CCS is run in parallel with --chunk option * 👌 IMPROVE: Add Pbindex in bam ouput channel * 👌 IMPROVE: Change label to process_low * 👌 IMPROVE: Define reports files names + add json version of txt report * 🐛 FIX: Add missing backslashes * 🐛 FIX: Add missing gz extension * 🐛 FIX: update ouput channel * 🐛 FIX: output file name * 👌 IMPROVE: .gitignore * 👌 IMPROVE: Update function.nf to last version * 👌 IMPROVE: Update saveAs in main.nf * 👌 IMPROVE: Add pbccs module * 🐛 FIX: Fix Broken test * 👌 IMPROVE: Update test_data.config * 🐛 FIX: Fix test * 👌 IMPROVE: Update path of test dataset files * 👌 IMPROVE: Remove useless index + Fix Typos * 🐛 FIX: fill contains args * 👌 IMPROVE: One output => One Channel * 👌 IMPROVE: One input => One channel * 🐛 FIX: Update tests * 🐛 FIX: Remove TODOs from test.yaml * 👌 IMPROVE: Revert and keep bam and pbi together * 🐛 FIX: Remove old rq input from meta.yml * 👌 IMPROVE: Update test to match input channels * 👌 IMPROVE: use prefix for for output file name * 👌 IMPROVE: Update to new versions.yml * 👌 IMPROVE: Update pbccs from v6.0.0 to v6.0.2 * 👌 IMPROVE: Keep track of the former sample id in meta * Update modules/pbccs/main.nf Co-authored-by: Harshil Patel * 👌 IMPROVE: remove former_id from meta * 👌 IMPROVE: Use chunk number in output filename * 🐛 FIX: Update meta.yml * 🐛 FIX: Update reports filenames with chunk number. Co-authored-by: James A. Fellows Yates Co-authored-by: Harshil Patel --- modules/pbccs/main.nf | 6 ++--- modules/pbccs/meta.yml | 22 +++++++++++++++--- ...t_versions_yml.cpython-39-pytest-6.2.5.pyc | Bin 0 -> 3558 bytes tests/modules/pbccs/test.yml | 10 ++++---- 4 files changed, 27 insertions(+), 11 deletions(-) create mode 100644 tests/__pycache__/test_versions_yml.cpython-39-pytest-6.2.5.pyc diff --git a/modules/pbccs/main.nf b/modules/pbccs/main.nf index 7e70ac14..55eacd76 100644 --- a/modules/pbccs/main.nf +++ b/modules/pbccs/main.nf @@ -37,9 +37,9 @@ process PBCCS { ccs \\ $bam \\ ${prefix}.chunk${chunk_num}.bam \\ - --report-file ${prefix}.report.txt \\ - --report-json ${prefix}.report.json \\ - --metrics-json ${prefix}.metrics.json.gz \\ + --report-file ${prefix}.chunk${chunk_num}.report.txt \\ + --report-json ${prefix}.chunk${chunk_num}.report.json \\ + --metrics-json ${prefix}.chunk${chunk_num}.metrics.json.gz \\ --chunk $chunk_num/$chunk_on \\ -j $task.cpus \\ $options.args diff --git a/modules/pbccs/meta.yml b/modules/pbccs/meta.yml index 38f31496..f55c0d71 100644 --- a/modules/pbccs/meta.yml +++ b/modules/pbccs/meta.yml @@ -42,10 +42,26 @@ output: type: file description: File containing software versions pattern: "versions.yml" - - css: + - bam: type: file - description: Consensus sequences - pattern: "*.ccs.bam" + description: CCS sequences in bam format + pattern: "*.bam" + - pbi: + type: file + description: PacBio Index of CCS sequences + pattern: "*.pbi" + - report_txt: + type: file + description: Summary of CCS in txt format + pattern: ".txt" + - report_json: + type: file + description: Summary of CCS in txt json + pattern: ".json" + - metrics: + type: file + description: Metrics about zmws + pattern: "*.json.gz" authors: - "@sguizard" diff --git a/tests/__pycache__/test_versions_yml.cpython-39-pytest-6.2.5.pyc b/tests/__pycache__/test_versions_yml.cpython-39-pytest-6.2.5.pyc new file mode 100644 index 0000000000000000000000000000000000000000..33acb8369a1bc62b5e66e1ed80e2247dd0e2759f GIT binary patch literal 3558 zcmaJ@TaVku73T0FQCdkWt=50fJ2%8xAU@;c_*j71tCg z&y04xD)|!Up)V=?+}g;h4+RSJp}(O&gXmj<0(mOXry?kto--7;v;&k9hjTgS%y%wx z=6LmbRfFfZUyVJ_*0jIjp!gUtco$v~KtnZ30msOtx zcHdT1m3~EitNki`jbLe9>(_wBq}G>Ozs~I4#_lq!Kh=Jt(Go3xsXZnACMEZ^j{Obn zsBLygYW~cVBlwmmqb!sIoX;N%I{4zG*n~!Ek~}55`k^t`5-rvaO-kk@Au+k9{qjxbpJ20feVU+IYv8?e?$%-E#Hn)VR1dVK5g8EOIwYxea|m~^}#a6)g)gcA+b zU_auIhC#G1oc)nM7&#+vk2#(rnULLsap3r&BS%b)g+qPL1~TFY-D=f==RxGq$Pv*c z2j-_-_K-ar?v6pkA zwLXeG%6Mub?#)hx>tLPKc3l+ex@ieSf{?V#nTUctmYO?3w4Is<-Z)6j0GQJXXC8GW zdm_{7d!AsK#EzAkBgR?(*yI5Bb8}lL$s#4g{*sypm_vUjwe$5qYPCkha|&Rsf(} z1YxX+8Z?^KAXQ@PtHdU}4dVg9*~N1VzDw}F53l&hz{rUq`Y>j8$)S!>L-jRH#SxjE zi6MGp@;iv!7shEV(IJLRh%n<*Equ#_SUUKeK->^bmSmY)u@01_7sgYF#V^P&v`}By zXc;3cHugwj#g<5=eORG(Yz+ym;QUaZn=n_6P1w0Amkw)lZBG-whq-!e%nARKT7_Py z(<)tht{>Nu63jJd4LIs2G}?$uK7rXXps){@6Z%*2w-!xl_&DfsB3@E@)E0VxerQLfhXuM+lZtH1)!05f zHV2=9CV(ieqeijyOc(DdX(vJ28b~Xm|0k^E=>Lrvj?kYvLa&RHNEO!sz0S`g74`xB zDb)w`^z?H>SwFLLDaQx<*AI`+HOMFb`b@Vuec(Dr&mU14LPx528K?|iQ}Un>RGHS`SvA)N*>5~AgT|{1kF4ezgEyM+w%}b! z+RyaIn^c3m`Q~OyT#n~$R*t4}GL^kqy_X+fOp&u~Oohc~tGK-7gpqWH(G-&3BAI4# zirKWtjs1-$6G*fS)1@5UcD82!*vc|)k?_ID7tX*7vz$Gi2GXAd3{tTtAhX<7Nwr9S z(Rr}QYCg>S(f z0T~~yAf(5ovk%!ZyHuhmyN$jtv|roUyyrgL_~ibFA8+38kZ()K?}r^SJN|WNX3Dpf5I(`sb z2IGu9iT?QM#wQ#1K7N1Jtlss6V4w|de*?CMoo;-CaROlVJ9hx+`T|+?SBs8wM^UeL z#08uI9Xi#-xs%P_L>U1KQ#%LMF;VC`KLJAdNoa6*bJka0j3$=@2G=d3%o)^$J)R1w zE>;ipV^6$+gey4;nFDtuKe(+7KjG1U39umg%ENnoAe~3-;1#uQWoq4;y$xGJcqq{j z44!lX2r{X}E;e`A*hdR>XWf@f1D^T+`ll@s4dp)E;YXKZ$KbD_&PR&RN}qp5-|k+V zwI62prV~!b+l<4p;?C@J2CNWV-eSG$%=k-au7K?`IM`eG<7z$!o&UM03mzYgs)=&d z^^UltdjBXLU1@A7C!o;5_1K`04o5_5YN<)4!|Mzp$$0FGiF6 z8dmuR&~{p>ojaWOU?N>}vuNpNn81Bq&h|MtCaoMzVR5bI_$A~i