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

Expose additional statistics in Picard Alignment Summary #1293

Closed
Redmar-van-den-Berg opened this issue Sep 14, 2020 · 2 comments · Fixed by #1294
Closed

Expose additional statistics in Picard Alignment Summary #1293

Redmar-van-den-Berg opened this issue Sep 14, 2020 · 2 comments · Fixed by #1294

Comments

@Redmar-van-den-Berg
Copy link
Contributor

Is your feature request related to a problem? Please describe.
The Picard Alignment Summary currently only shows the number of aligned reads in the barchart. This makes sense for Illumina data, where all reads are the same size. However, for long read sequencing, the number of aligned reads is not so relevant, and the number of aligned bases or the mean read size is more relevant.

Describe the solution you'd like
Add additional datasets to the picard Alignment Summary section. This way, the default behaviour will be the same (show the number of aligned reads), but users can switch over to number of aligned bases or mean read length, if this is more relevant for them.

Describe alternatives you've considered
I was also considering adding additional Read based statistics to the graph but making them invisible by default (like they are when you click on the legend in the HTML report), but I could not find a way to do this in the documentation.

@ewels
Copy link
Member

ewels commented Dec 28, 2020

There could also be a config option for this to allow it to be set as default when running routinely with this kind of data.

@Redmar-van-den-Berg
Copy link
Contributor Author

I'm not sure if a configuration setting to change the default is a good solution. Even for long reads, you still want to have an idea of the number of aligned reads. Having buttons to switch to the additional data is also easier to use then having a config option.

Depending on the experimental setup, having the mean read length and total number of aligned bases might also be relevant for people working with Illumina data, so it would be nice to have this data available for all users.

@ewels ewels linked a pull request Mar 7, 2021 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants