feat: Add markdown script for CNV report generation#89
Conversation
PR checklist
See Contributing Guidelines for more details on how to contribute according to pipeline conventions. |
There was a problem hiding this comment.
Note: Changelog will be updated as part of parent issue
There was a problem hiding this comment.
Even though the CI checks might fail, you should still be able to merge to your feature branch since it's not a protected branch! 😉
|
|
||
|
|
||
| <details> | ||
| <summary><strong>How to use the yables (click to expand)</strong></summary> |
There was a problem hiding this comment.
| <summary><strong>How to use the yables (click to expand)</strong></summary> | |
| <summary><strong>How to use the tables (click to expand)</strong></summary> |
There was a problem hiding this comment.
If this script is to be used within a module, I would put this script within the module: https://docs.seqera.io/nextflow/modules/developing-modules#resources 😊
There was a problem hiding this comment.
I'm curious: How would this work if we were to use a shared module, like nf-co.re/modules/rmarkdownnotebook? 🤔
Do you have any idea @fellen31 ?
There was a problem hiding this comment.
Not sure...old way of doing things was to put all scripts in a shared ./bin/ folder.
There was a problem hiding this comment.
Ah it seems that the module itself receives the path to the file as an input, so I guess it doesn't really matter where the file is located.
But otherwise, putting it in a shared ./bin/ directory would probably be the way to go since nextflow will automatically detect it there, as far as I understand!
There was a problem hiding this comment.
Well, it's only if the script is to be used within the script: block (and not as an input file). If it is actually an input, then I guess assets is fine.
| --- | ||
|
|
||
| ```{r setup, include=FALSE} | ||
| library(readr) |
| **Filtering** - Use the boxes at the top of each column to filter results. The tables support **regular expression (regex)** search: | ||
|
|
||
| - Exact match (recommended for genes): `^TP53$` → matches only TP53 | ||
| - Starts with: `^BRCA` | ||
| - Ends with: `TP53$` |
|
A small thing I noticed when looking at the example report was that the Gene column search was a little buggy. The filtering seemed to work but the searchbar didn't display the genename you search for. But maybe that can be addressed later! |
This script is part of https://github.com/Clinical-Genomics/MTP-oncoflow/issues/34
Added
Notes
This PR displays tables as-is. The report will need to be optimised once we know which values are relevant to display after assessing the verification cases.
Standalone script usage: