search

qiime2 / q2-demux

BSD 3-Clause "New" or "Revised" License
0 stars 20 forks source link

add seven-number summary of read length distribution to `summarize` #71

Closed gregcaporaso closed 6 years ago

gregcaporaso commented 7 years ago

This is useful after read joining, as the reads will differ in length, and this information is essential to have for setting the trim length parameter for deblur denoise-*.

If you need convincing of why we should add this, check out how awful the Viewing summary of joined data with read quality section of the read joining community tutorial has to be without it.

gregcaporaso commented 6 years ago

While making this change, let's also add the number of samples (maybe in the same table). I had to copy/paste the seqs/sample counts data into a text file to figure out the number of samples in one of these artifacts to track down an issue I was having with an analysis.

julibeg commented 3 years ago

As far as I can tell, the current version of the read-joining tutorial still describes the old plot and does not mention the seven-number summary. It also notes "We plan to simplify this process in the near future." referencing this issue. I guess it has not been updated yet.

andrewsanchez commented 3 years ago

Hi, @julibeg! The 7-number summary is on the interactive quality plot tab of the visualization output by qiime demux summarize. But you are correct, this tutorial needs to be updated. Thanks for the report! I created a new issue in the qiime2/docs repo (qiime2/docs#503). Would you be interested in submitting a pull request to update this section of the tutorial?

julibeg commented 3 years ago

I just started using qiime. Thus, I guess it's better if more experienced users handle the tutorials.