Re: [Denovoassembler-devel] Ray-Cloud-Browser

[Sébastien B. <seb...@ul...>]

On Fri, Jan 11, 2013 at 02:11:02PM -0500, Pier-Luc Plante wrote:
> Salut,
> j'ai approfondi mon analyse du bogue...
> Pour un nombre (ex:12345), il est impossible d'aller plus haut que la
> valeur maximal du chiffre à une position donnée.
> Donc, par exemple, la position 999 ne peut pas être sélectionnée pour
> une longueur de contig de 12345 mais pourtant la position existe.

Obviously that's a bug.

> Je suggère donc de mettre le minimum à 0 et le maximum à 9 excepté pour
> les IntegerSelectionWidget critiques de gauche.
> Par exemple pour un contig d'une longueur de 12345,
> au stade initial:
> Pos1--> 0..9
> Pos2--> 0..9
> Pos3--> 0..9
> Pos4--> 0..9
> Pos5--> 0..1
> 
> Si en cours d'utilisation le nombre sélectionné se trouve à devenir
> 12000, les Widgets pourront:
> Pos1--> 0..9
> Pos2--> 0..9
> Pos3--> 0..3
> Pos4--> 0..2
> Pos5--> 0..1
> Il faudra donc faire vérifier chaque Widget à droite de celui qui est
> modifié pour s'assurer qu'il n'est pas à une valeur impossible.
> Sinon je vois toujours l'option de permettre le 0..9 pour tout les
> widget et de faire afficher "Position impossible" si la position
> n'existe pas.
> Qu'est-ce que tu en penses?

I agree. Can you try this patch which solves the issue ?

	patch -p1 < fix-range.patch

If you think it solves the bug, commit that on your repository and add
my signed-off line like this in your commit:

Signed-off-by: Pier-Luc Plante <pie...@ul...>
Signed-off-by: Sébastien Boisvert <seb...@ul...>

Then send the pull request here as usual.

Thanks.

> Je peux essayer de le coder si tu veux...
> 
> Pier-Luc