bioforensics / MicroHapulator

Tools for empirical microhaplotype calling, forensic interpretation, and simulation.
https://microhapulator.readthedocs.io/
Other
6 stars 1 forks source link

Replace donut plots with stacked bar graph #168

Closed standage closed 8 months ago

standage commented 8 months ago

Currently, a breakdown of reads by category (on target, off target, repetitive, and contamination) is displayed in the report as a grid of donut plots, one per sample. This PR replaces these donut plots with a single stacked bar chart. It is shown in horizontal orientation so that the height of the plot can grow as the number of samples grows.

read-mapping-qc

Related to #160.


standage commented 8 months ago

This is how the plot looks in context. Screenshot 2024-03-18 at 11 51 26 AM

agshumate commented 8 months ago

Code looks good but legend placement looks a little funky with a small number of samples

image
standage commented 8 months ago

An additional line of code seems to have addressed this issue, and appears to work as intended regardless of the number of samples.


read-mapping-qc


read-mapping-qc


read-mapping-qc

agshumate commented 8 months ago

Looks good!