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

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

Thanks for looking into this, I merged your work.

On Sat, Jan 12, 2013 at 12:20:00PM -0500, Pier-Luc Plante wrote:
> 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
>