-
Notifications
You must be signed in to change notification settings - Fork 17
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
report #10
Comments
We can base the report on the following script: and modify it according to the Nanopore needs. |
@oliverdrechsel @rekm welcome to the reporting issue! :) So basically @replikation needs to know what are the inputs (and formats) for the different parts https://gitlab.com/RKIBioinformaticsPipelines/ncov_minipipe/-/blob/master/ncov_minipipe.Rmd already provides so he can generate the inputs. Maybe we can start basic and try to implement the
|
yep basically either an example "of input" to this script or a summary of what the script needs to function properly |
the report currently takes a bunch of files. I think we can either modify the report to have all files optional or butcher it and make a nanopore version What i find quite important for the report output (html) is that figures and tables have a maximum size and implement scroll bars. This keeps the report short and easy to scroll, although each section might contain 100 samples. *coverage.tsvloads of 0's because it's a negative control as example
fragment sizethis is basically the fragment size column of the mapping bam file
mapping statisticsoutput from bwa mem
transformed for improved reading in R
version
pangolin (optional)
kraken (optional)kraken2 result of a run of reads against a human/SARS-CoV-2 database (https://zenodo.org/record/3854856)
|
@oliverdrechsel thanks ill prepare the information on my end and check if I can add all the mandatory items so you don't need to change anything. |
Thanks a lot. I don't know if things like fragment size make sense as minION is not performing paired end sequencing. |
@oliverdrechsel just checked its 0 everywhere. so this might be good as an optional parameter or do you have another idea that I can supply here instead? |
@oliverdrechsel looking at your report script i tend to rewrite it as nextflow is more "file" oriented. so using dirs and recursively looking for inputs is counter intuitive here. Would this be okay if I fork your script and adjust it to the workflow? |
It would be good to have at least a simple summary report for the reconstructed consensus sequence. This should include:
E.g. in a single PDF report per run.
For the first part (technical stats) it might be also enough to use the nextflow internal functions for reporting.
The text was updated successfully, but these errors were encountered: