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

Re-branding nf-seqqc to bacpaq #61

Merged
merged 41 commits into from
May 31, 2024
Merged

Re-branding nf-seqqc to bacpaq #61

merged 41 commits into from
May 31, 2024

Conversation

anwarMZ
Copy link
Member

@anwarMZ anwarMZ commented Apr 8, 2024

No description provided.

@anwarMZ
Copy link
Member Author

anwarMZ commented Apr 8, 2024

@jimmyliu1326 there is a lot of information here - let's try to go over it in this thread and solve it sequentially.

@jimmyliu1326
Copy link
Collaborator

jimmyliu1326 commented Apr 9, 2024

Documenting discussion points as I go through the PR:

  • Dehosting is removing all the bacterial reads
  • The recommended method to run the workflow is by referencing the Github release i.e. nextflow run cidgoh/bac-pac -r [release]
  • Consistent pipeline naming bacpaq or bac-paq
  • ERROR ~ Failed to invoke workflow.onComplete event handler
  • Sample sheet validation scheme schema_input.json is incompatible with assembled genome and nanopore data inputs
  • Missing parameters in nextflow_schema.json e.g. workflow
  • Printing pipeline version requires user to supply a valid profile

@jimmyliu1326
Copy link
Collaborator

jimmyliu1326 commented May 11, 2024

Keeping track of the progress on the aforementioned issues:

  • Dehosting is removing all the bacterial reads c96a7ea
  • The recommended method to run the workflow is by referencing the Github release i.e. nextflow run cidgoh/bac-pac -r [release]
  • Consistent pipeline naming bacpaq or bac-paq
  • ERROR ~ Failed to invoke workflow.onComplete event handler
  • Sample sheet validation scheme schema_input.json is incompatible with assembled genome and nanopore data inputs cfbe66e
  • Missing parameters in nextflow_schema.json e.g. workflow
  • Printing pipeline version requires user to supply a valid profile a6dd605
  • File name conflict between Illumina and Nanopore genome assembly when publishing to output directory

Copy link

github-actions bot commented May 31, 2024

nf-core lint overall result: Failed ❌

Posted for pipeline commit 9cb326c

+| ✅ 223 tests passed       |+
!| ❗  84 tests had warnings |!
-| ❌  86 tests failed       |-

❌ Test failures:

  • files_exist - File must be removed: lib/Utils.groovy
  • files_exist - File not found check: lib/nfcore_external_java_deps.jar
  • nextflow_config - Config variable not found: params.validationShowHiddenParams
  • nextflow_config - Config variable not found: params.validationSchemaIgnoreParams
  • nextflow_config - Config manifest.name did not begin with nf-core/:
    cidgoh/bacpaq
  • nextflow_config - Config variable manifest.homePage did not begin with https://github.com/nf-core/:
    https://github.com/cidgoh/bacpaq
  • nextflow_config - Config default value incorrect: params.mode is set as illumina in nextflow_schema.json but is nanopore in nextflow.config.
  • nextflow_config - Config default value incorrect: params.skip_subsampling is set as true in nextflow_schema.json but is false in nextflow.config.
  • nextflow_config - Config default value incorrect: params.depth_cut_off is set as 50,80 in nextflow_schema.json but is 50 in nextflow.config.
  • nextflow_config - Default value from the Nextflow schema params.tracedir = 25.MB`` not found in nextflow.config.
  • nextflow_config - Config default value incorrect: params.validate_params is set as true in nextflow_schema.json but is false in nextflow.config.
  • nextflow_config - Config default value incorrect: params.schema_ignore_params is set as fasta in nextflow_schema.json but is genomes in nextflow.config.
  • files_unchanged - .github/CONTRIBUTING.md does not match the template
  • files_unchanged - .github/ISSUE_TEMPLATE/bug_report.yml does not match the template
  • files_unchanged - .github/ISSUE_TEMPLATE/feature_request.yml does not match the template
  • files_unchanged - .github/PULL_REQUEST_TEMPLATE.md does not match the template
  • files_unchanged - .github/workflows/branch.yml does not match the template
  • files_unchanged - .github/workflows/linting_comment.yml does not match the template
  • files_unchanged - .github/workflows/linting.yml does not match the template
  • files_unchanged - assets/email_template.html does not match the template
  • files_unchanged - assets/email_template.txt does not match the template
  • files_unchanged - assets/sendmail_template.txt does not match the template
  • files_unchanged - docs/README.md does not match the template
  • actions_ci - Minimum pipeline NF version '22.01.0' is not tested in '.github/workflows/ci.yml'
  • readme - README Nextflow minimum version badge does not match config. Badge: 23.04.0, Config: 22.01.0
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/modules/local/RMLST/ubuntu_curl_test.sif L319578: b{{XmwçDç�Ë{«=4�¼Ö¶?fÍÛp�Õ}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-angular.mjs L2: "{{","}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-angular.mjs L2: "{{",I="}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L6: "{{{",e.strip.open&&(this.buffer+=""),this.Expression(e.path),this.Params(e.params),this.Hash(e.hash),e.strip.close&&(this.buffer+=""),this.buffer+=e.escaped?"}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L6: "{{#",this.Expression(e.path),this.Params(e.params),this.Hash(e.hash),e.program.blockParams.length&&this.BlockParams(e.program.blockParams),e.chained?this.buffer+=e.inverseStrip.close?"~}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L6: "{{",this.buffer+="else",this.buffer+=e.inverseStrip.close?"~}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L6: "{{/",this.Expression(e.path),this.buffer+=e.closeStrip.close?"~}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L6: "{{>",this.Expression(e.name),this.Params(e.params),this.Hash(e.hash),this.buffer+="}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L6: "{{",this.Expression(e.path),this.Params(e.params),this.Hash(e.hash),this.buffer+="}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L12: "{{ "+this.SubExpression(l)+" }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L12: "{{ DIRECTIVE "+this.SubExpression(l)+" }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L12: "{{^}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L12: "{{> "+e+" }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L12: "{{> "+e+" }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L12: "{{! '"+l.value+"' }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L26: `{{${(0,c.printLiteral)(b)}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L27: ({{> ...}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L27: `{{#${M.path}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-glimmer.mjs L27: `{{#${C.path}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-html.mjs L34: "{{","}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-html.mjs L35: "{{ '{' }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/parser-markdown.mjs L75: |{{.*?}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/standalone.mjs L75: "{{",Re(H,X),"}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/standalone.mjs L107: /{{(.+?)}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/standalone.mjs L110: /{{(.+?)}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/esm/standalone.mjs L110: "{{",d(Z),"}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/index.js L32245: "{{", printPathAndParams(path, print2), "}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/index.js L35748: /{{(.+?)}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/index.js L36588: /{{(.+?)}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/index.js L36602: "{{", replaceTextEndOfLine(part), "}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-angular.js L2: "{{","}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-angular.js L2: "{{",I="}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L6: "{{{",e.strip.open&&(this.buffer+=""),this.Expression(e.path),this.Params(e.params),this.Hash(e.hash),e.strip.close&&(this.buffer+=""),this.buffer+=e.escaped?"}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L6: "{{#",this.Expression(e.path),this.Params(e.params),this.Hash(e.hash),e.program.blockParams.length&&this.BlockParams(e.program.blockParams),e.chained?this.buffer+=e.inverseStrip.close?"~}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L6: "{{",this.buffer+="else",this.buffer+=e.inverseStrip.close?"~}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L6: "{{/",this.Expression(e.path),this.buffer+=e.closeStrip.close?"~}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L6: "{{>",this.Expression(e.name),this.Params(e.params),this.Hash(e.hash),this.buffer+="}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L6: "{{",this.Expression(e.path),this.Params(e.params),this.Hash(e.hash),this.buffer+="}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L12: "{{ "+this.SubExpression(o)+" }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L12: "{{ DIRECTIVE "+this.SubExpression(o)+" }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L12: "{{^}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L12: "{{> "+e+" }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L12: "{{> "+e+" }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L12: "{{! '"+o.value+"' }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L26: `{{${(0,c.printLiteral)(b)}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L27: ({{> ...}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L27: `{{#${M.path}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-glimmer.js L27: `{{#${C.path}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-html.js L34: "{{","}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-html.js L35: "{{ '{' }}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/parser-markdown.js L75: |{{.*?}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/standalone.js L75: "{{",Re(H,X),"}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/standalone.js L107: /{{(.+?)}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/standalone.js L110: /{{(.+?)}}
  • template_strings - Found a Jinja template string in /home/runner/work/nf-seqqc/nf-seqqc/node_modules/prettier/standalone.js L110: "{{",y(Z),"}}
  • schema_params - Default value for param skip_subsampling invalid: Schema default (True) does not match the config default (false)
  • schema_params - Default value for param tracedir invalid: Not in pipeline parameters. Check nextflow.config.
  • schema_params - Default value for param validate_params invalid: Schema default (True) does not match the config default (false)
  • multiqc_config - assets/multiqc_config.yml does not meet requirements: Section cidgoh-bacpaq-summary missing in report_section_order
  • modules_config - conf/modules.config contains withName:MINIMAP2_ILLUMINA, but the corresponding process is not present in any of the Nextflow scripts.
  • modules_config - conf/modules.config contains withName:MINIMAP2_NANOPORE, but the corresponding process is not present in any of the Nextflow scripts.

❗ Test warnings:

✅ Tests passed:

Run details

  • nf-core/tools version 2.14.1
  • Run at 2024-05-31 21:13:20

@anwarMZ anwarMZ merged commit ee42604 into dev May 31, 2024
0 of 4 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