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

[Pier-Luc P. <pie...@ul...>]

The patch you proposed solved the problem.
Please pull from me.

      git://github.com/plpla/Ray-Cloud-Browser.git               master

Thanks
Pier-Luc

Le 2013-01-11 15:07, Sébastien Boisvert a écrit :
> 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