search

Sung-Huan / ANNOgesic

ANNOgesic - A Swiss army knife for the RNA-Seq based annotation of bacterial/archaeal genomes
http://annogesic.readthedocs.io/en/latest/index.html
Other
30 stars 13 forks source link

Merging results srna_target, wrong column names #24

Closed gprezza closed 3 years ago

gprezza commented 3 years ago

Hi!

I think I found a small bug when merging results from different methods of srna_target:

https://github.com/Sung-Huan/ANNOgesic/blob/7b136ae125eef47a2e9e16b4ca90f5766ecb0570/annogesiclib/merge_rnaplex_rnaup.py#L462 https://github.com/Sung-Huan/ANNOgesic/blob/7b136ae125eef47a2e9e16b4ca90f5766ecb0570/annogesiclib/merge_rnaplex_rnaup.py#L463

I guess these two lines should instead be:

 "Energy_" + methods[0], "Rank_" + methods[0], 
 "Energy_" + methods[1], "Rank_" + methods[1]]) + "\n")

(please note the indexes of methods)

Sung-Huan commented 3 years ago

Thank you for your information. Yes, it is a bug. I will find time to fix it.

Sung-Huan commented 3 years ago

The bug was fixed in the latest version.