Tiny bug in the reference implementation (sequence parsing)

Tiny bug in the reference implementation (sequence parsing)


I've noticed a tiny bug in the reference implementation:
if a sequence has no name, eg. just "AAAA...", the first letter is not read into the sequence.

This is because of the first s.get() in getSequence:


and can be fixed by adding


below it.


4 posts / 0 nouveau(x)
Dernière contribution
Reportez-vous à notre Notice d'optimisation pour plus d'informations sur les choix et l'optimisation des performances dans les produits logiciels Intel.

We followed the reference implementation and implemented this buggy behaviour.

It would be interesting if we should fix it.

To my mind, the case you describe should not be encountered.It does not correspond to a correct input.

The guys from Intel always state that they will provide only valid and well formed files. I do not think they will be that picky and test for special cases like this. All that counts is a fast implementation that scales and not an implementation that imitates bugs of the reference implementation.

But thank you very much for that advice, now that I know that special behavior, I will see if I can get it into our code as well ;-)

Laisser un commentaire

Veuillez ouvrir une session pour ajouter un commentaire. Pas encore membre ? Rejoignez-nous dès aujourd’hui