semantic confusion in stats

[sergpolly]

We used to have following fields in the stats before https://github.com/mirnylab/pairsamtools/commit/85ddfb309591cb0b8bea56130ca5648afeac77d6

total_unmapped
total_single_sided_mapped
dedup/n_unmapped

Now after https://github.com/mirnylab/pairsamtools/commit/85ddfb309591cb0b8bea56130ca5648afeac77d6 we are collecting stats in the dedup step the same way, as before is parse, so this specialized field

dedup/n_unmapped

is gone now.

What's confusing, to me at least, is to have a field total_unmapped in stats and have there - "unmapped on both sides" ...

Should we rename it to: total_both_sides_unmapped as suggested by @golobor , or something like that. Or should we make total_unmapped to include both total_both_sides_unmapped and total_single_sided_mapped ? @nvictus ?

[sergpolly]

here's what we need to implement:

[agalitsyna]

@sergpolly Is it still relevant? Seems like we are okay with current version, which is an input to multiqc. Changing the pairs standards will need to modify multiqc as well.