Closed WackerO closed 4 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit e393cdd
+| ✅ 273 tests passed |+
!| ❗ 3 tests had warnings |!
Assume the GSEA changes here are because we need to merge in master.
Actually. what I was thinking with the warning is to actually check the data for values of 0 in the p values, and conditionally display a prominent warning under the plot:
"x features with p values of 0 are not shown- please refer to the results tables"
Ah, those are in this PR because I created volcano_round from dev after adding the GSEA update.
Can also do that.
How about this? @pinin4fjords SRP254919 2.html.zip
Instead of https://github.com/nf-core/differentialabundance/pull/229
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).