search

wdecoster / nanocomp

Comparison of multiple long read datasets
MIT License
113 stars 9 forks source link

Bar labels swapped for 'total_throughput' graph #58

Closed cdebuck closed 2 years ago

cdebuck commented 2 years ago

Hello, I've used NanoComp v1.19 both with summary files and fastq files (each time with a pair of input files). I've used the -n parameter to give names to my samples. I noticed that the graph called 'comparing throughput in bases' in the NanoComp report (= NanoComp_total_throughput.png) assigns the wrong labels to the bars, because it does not match the numbers from the NanoStats.txt file. Also for the 'number of reads' graph, the labeling is correct, but the order of the bars is swapped compared to the other graphs, making it harder to interpret (because it first reports 'name-label-2' and then 'name-label-1'.

The command I've used is: NanoComp --summary sum_file_1 sum_file_2 -t 16 -n guppy_102 bonito_102 -o outdir

NanoComp_total_throughput NanoComp_number_of_reads NanoStats.txt

wdecoster commented 2 years ago

Hi,

Thanks for the report, that should be fixed in NanoComp v1.19.1, which I just pushed to PyPI and should available through conda within a few hours.

Wouter