wurmlab / sequenceserver

Intuitive graphical web interface for running BLAST bioinformatics tool (i.e. have your own custom NCBI BLAST site!)
https://sequenceserver.com
GNU Affero General Public License v3.0
274 stars 114 forks source link

Fix: reformat long seqs #806

Open joko3ono opened 1 month ago

joko3ono commented 1 month ago

resolve wurmlab/sequenceserver-cloud#415

djleite commented 1 month ago

things look almost ok

however, there needs to be correct rounding up of the final sequence length. See below that:

image image

However...

If we want to show a greater level of precision then we can show to at least 1 decimal place for any value that has units kbp, Mbp and Gbp.

ie divide these numbers by 1000 and round to the nearest decimal place

divide these numbers by 1000000 and round to the nearest decimal place

tadast commented 1 month ago

@joko3ono also please make sure you add tests for the rounding so that it's not broken in future