Skip to content

Deseq pathway species#104

Merged
WackerO merged 16 commits intoqbic-pipelines:devfrom
WackerO:deseq_pathway_species
Feb 17, 2022
Merged

Deseq pathway species#104
WackerO merged 16 commits intoqbic-pipelines:devfrom
WackerO:deseq_pathway_species

Conversation

@WackerO
Copy link
Copy Markdown
Contributor

@WackerO WackerO commented Feb 16, 2022

Many thanks to contributing to qbic-pipelines!

This PR closes issues #71 and #102 and possibly also #91 by making pathway analysis optional (just set skip_pathway_analysis = true). If skipped, no error will be thrown when omitting the species parameter.

PR checklist

  • This comment contains a description of changes (with reason)
  • If you've fixed a bug or added code that should be tested, add tests!
  • Ensure the test suite passes (nextflow run . -profile test_skip_pathway_analysis,docker).
  • Documentation in docs is updated
  • CHANGELOG.md is updated
  • README.md is updated

 Zum Commit vorgemerkte Änderungen:
	geändert:       bin/DESeq2.R
	geändert:       conf/test.config
	geändert:       nextflow.config
	geändert:       workflows/rnadeseq.nf
	-->added parameter --skip_pathway_analysis; DESeq2.R needs to be rolled back before pushing!
 Zum Commit vorgemerkte Änderungen:
	geändert:       assets/RNAseq_report.Rmd
	geändert:       bin/DESeq2.R
	geändert:       conf/test.config
	geändert:       modules/local/report.nf
	geändert:       workflows/rnadeseq.nf
	-->started with the optional PA and making species optional; Susi's fix in threshold is also included in DESeq2.R
Merging deseq_previous_code into deseq_pathway_species to apply the most current DSL2 porting changes to side branch
 Zum Commit vorgemerkte Änderungen:
	geändert:       assets/RNAseq_report.Rmd
	geändert:       bin/Execute_report.R
	geändert:       modules/local/report.nf
	-->trying to make the pathway analysis skippable
 Zum Commit vorgemerkte Änderungen:
	geändert:       bin/Execute_report.R
	geändert:       conf/test.config
	geändert:       modules/local/report.nf
	-->PA is skippable!
 Zum Commit vorgemerkte Änderungen:
	geändert:       assets/RNAseq_report.Rmd
	geändert:       conf/test.config
	-->trying to fix missing "enriched pathways" subheader
 Zum Commit vorgemerkte Änderungen:
	geändert:       .github/workflows/ci.yml
	geändert:       assets/RNAseq_report.Rmd
	geändert:       bin/DESeq2.R
	neue Datei:     conf/test_no_pathway_analysis.config
	geändert:       nextflow.config
	-->fixed report issues, added conf file for testing the pipeline with skipped pathway analysis
 Ihr Branch ist auf demselben Stand wie 'origin/deseq_pathway_species'.

 Zum Commit vorgemerkte Änderungen:
	geändert:       CHANGELOG.md
	neue Datei:     conf/test_skip_pathway_analysis.config

 Änderungen, die nicht zum Commit vorgemerkt sind:
	gelöscht:       conf/test_no_pathway_analysis.config
	-->renamed file
 Ihr Branch ist auf demselben Stand wie 'origin/deseq_pathway_species'.
 Zum Commit vorgemerkte Änderungen:
	geändert:       bin/DESeq2.R
	geändert:       modules/local/report.nf
	geändert:       nextflow_schema.json
 Ihr Branch ist 1 Commit vor 'origin/deseq_pathway_species'.
   (benutzen Sie "git push", um lokale Commits zu publizieren)

 Zum Commit vorgemerkte Änderungen:
	gelöscht:       conf/test_no_pathway_analysis.config
 Ihr Branch ist auf demselben Stand wie 'origin/deseq_pathway_species'.
 Zum Commit vorgemerkte Änderungen:
	geändert:       .github/workflows/ci.yml
	geändert:       nextflow.config
 Ihr Branch ist auf demselben Stand wie 'origin/deseq_pathway_species'.

 Zum Commit vorgemerkte Änderungen:
	geändert:       assets/RNAseq_report.Rmd
	-->fixed linting error with whitespace
 Ihr Branch ist auf demselben Stand wie 'origin/deseq_pathway_species'.

 Zum Commit vorgemerkte Änderungen:
	geändert:       assets/RNAseq_report.Rmd
	gelöscht:       assets/report_options.yml
	-->removed assets/report_options.yml and the KEGG option from assets/RNAseq_report.Rmd; only --skip_pathway_analysis is now used
 Ihr Branch ist auf demselben Stand wie 'origin/deseq_pathway_species'.

 Zum Commit vorgemerkte Änderungen:
	geändert:       assets/RNAseq_report.Rmd
	geändert:       bin/Execute_report.R
	geändert:       modules/local/report.nf
	geändert:       workflows/rnadeseq.nf
 Ihr Branch ist auf demselben Stand wie 'origin/deseq_pathway_species'.

 Zum Commit vorgemerkte Änderungen:
	geändert:       CHANGELOG.md
Copy link
Copy Markdown
Collaborator

@ggabernet ggabernet left a comment

Choose a reason for hiding this comment

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

Hi @WackerO ,

the PR looks good to me, thanks!

Comment thread nextflow_schema.json
@WackerO WackerO merged commit abda87f into qbic-pipelines:dev Feb 17, 2022
@WackerO WackerO deleted the deseq_pathway_species branch February 24, 2022 09:24
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