merenlab / anvio

An analysis and visualization platform for 'omics data
http://merenlab.org/software/anvio
GNU General Public License v3.0
413 stars 142 forks source link

Inversions activity issue #2198

Closed FlorianTrigodet closed 5 months ago

FlorianTrigodet commented 5 months ago

I added a check if activity is missing for one oligo_primer (first or second). The main fix is the second commit. We need to always have an entry for the reference oligo in the activity table. And if there is no activity (short reads) matching the reference oligo, we create an entry with frequency 0. We also have a filter with a min frequency value to report the activity (avoid very rare and likely irrelevant oligo). In rare cases, a ref oligo can have a frequency below the threshold and is not included in the final table, and no entry with frequency 0 is created. The html part of the code expect an entry for the ref oligo. I corrected and tested the fix and it works 🎉

meren commented 5 months ago

Excellent! :)