vidjil issueshttps://gitlab.inria.fr/vidjil/vidjil/-/issues2023-03-01T17:04:11+01:00https://gitlab.inria.fr/vidjil/vidjil/-/issues/2042Conserver l'url suite à un login2023-03-01T17:04:11+01:00Mathieu GiraudConserver l'url suite à un loginSi j'accède à une URL de type http://app.vidjil.org/browser/index.html?patient=1803&config=35 sans être loggué, je tombe sur la page de login.
Après le login, ce serait bien d'accéder directement à la page voulue. Ce n'est pas le cas act...Si j'accède à une URL de type http://app.vidjil.org/browser/index.html?patient=1803&config=35 sans être loggué, je tombe sur la page de login.
Après le login, ce serait bien d'accéder directement à la page voulue. Ce n'est pas le cas actuellement, au moins pour les comptes admin.
@mikael-sRyan HerbertRyan Herberthttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2041L'alignement multiple plante quand la première séquence est retirée2019-04-03T14:23:12+02:00Mikaël SalsonL'alignement multiple plante quand la première séquence est retiréeAllons sur le [jeu de démo](http://rbx.vidjil.org/browser/index.html?patient=68&config=32). On sélectionne tous les clones en TRGV2–TRGJ1. On supprime le premier de la sélection et on aligne le tout. ~~L'alignement devient complètement f...Allons sur le [jeu de démo](http://rbx.vidjil.org/browser/index.html?patient=68&config=32). On sélectionne tous les clones en TRGV2–TRGJ1. On supprime le premier de la sélection et on aligne le tout. ~~L'alignement devient complètement foireux à partir du N, avec quasiment tous les nucléotides qui sont soulignés.~~
~~Il semble y avoir deux bugs (dans l'alignement CGI, puisque l'alignement devient foireux) et dans le JS, puisque la coloration est foireuse. Mais je penche pour une explication plus parcimonieuse d'un seul bug qui causerait les deux problèmes.~~
@magiraud @RyanHerbhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2037Pagination de la requête patient/sample_set2017-07-10T17:01:30+02:00Ryan HerbertPagination de la requête patient/sample_setLa requête patient est lente (#1542).
Une solution est de paginer les résultats.
Les problèmes relevés sont que les données supplémentaires sont calculés dans des requêtes secondaires, et donc les fonctions de filtre et de sort ne ...La requête patient est lente (#1542).
Une solution est de paginer les résultats.
Les problèmes relevés sont que les données supplémentaires sont calculés dans des requêtes secondaires, et donc les fonctions de filtre et de sort ne peuvent pas fonctionner correctement avec la pagination.
La solution proposée était de désactiver la pagination lors du filtre ou du sort des résultats
@RyanHerb @magiraud @mikael-s Web 2017.03Ryan HerbertRyan Herberthttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2035Colonne "results" de la liste patient vide2023-11-09T11:04:13+01:00Thonier FlorianColonne "results" de la liste patient videSuite au message de Aurélie ~"LIL-Lille" :
pb : colonne "results" vide lors de l'affichage de la liste des patients, pour certains patients dont l'analyse est lancé.
Bug non reproductible, mais en fouillant un peu plus, j'ai remarqué ...Suite au message de Aurélie ~"LIL-Lille" :
pb : colonne "results" vide lors de l'affichage de la liste des patients, pour certains patients dont l'analyse est lancé.
Bug non reproductible, mais en fouillant un peu plus, j'ai remarqué que les patients concernés devaient avoir une analyse lancée mais non finalisée (ou non commencée), donc sans résultats disponibles lors de sa requete.
Solution (voir #2036) : Ajouter une valeur dans la colonne si une analyse est en cours ou programmé (ex : "clonalité (computing)" ou "waiting" )
Cette valeur devrait etre grisée ou formattée pour la mettre en évidence par rapport aux autres, de plus, ne pas etre cliquable.
@magiraud @mikael-s @RyanHerbhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2034Format JSON pour réponse2022-07-01T11:49:44+02:00Mathieu GiraudFormat JSON pour réponseSuite à #2032, devrait-on avoir un standard lorsqu'on renvoie du JSON ? Un "status" ou un "data" quand tout est bon ?
http://stackoverflow.com/questions/12806386/standard-json-api-response-format
@mikael-s @RyanHerbSuite à #2032, devrait-on avoir un standard lorsqu'on renvoie du JSON ? Un "status" ou un "data" quand tout est bon ?
http://stackoverflow.com/questions/12806386/standard-json-api-response-format
@mikael-s @RyanHerbhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2032Modifier le contrôleur du segmenteur pour permettre l'interaction avec la pag...2017-04-12T14:07:43+02:00Vidjil TeamModifier le contrôleur du segmenteur pour permettre l'interaction avec la page d'ArmandDiscussion avec @RyanHerb et @Tydax
La page d'accueil est statique, pas besoin de passer par un contrôleur. En revanche, l'envoi des séquences doit faire appel en AJAX au contrôleur qui nous renvoie un JSON qu'on peut donner à manger au...Discussion avec @RyanHerb et @Tydax
La page d'accueil est statique, pas besoin de passer par un contrôleur. En revanche, l'envoi des séquences doit faire appel en AJAX au contrôleur qui nous renvoie un JSON qu'on peut donner à manger au modèle, via parseJsonData.
Il faut donc modifier le contrôleur : c'est lui qui gérait le formulaire, c'est inutile maintenant et renvoyer un JSON.
Poke @magiraud
***
@magiraud @RyanHerb @mikael-shttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2029"Color by tag" ne fonctionne pas après un autre "Color by" dans la liste des ...2016-11-29T14:43:25+01:00Vidjil Team"Color by tag" ne fonctionne pas après un autre "Color by" dans la liste des cloneshttp://rbx.vidjil.org/browser/?patient=1507&config=25
-> Color by J (ou by locus, encore mieux- -> Color by tag
On ne peut pas revenir au color by tag.
***
Pas de problème chez moi, ou je n'ai pas compris le problème.
***
J'avais mal p...http://rbx.vidjil.org/browser/?patient=1507&config=25
-> Color by J (ou by locus, encore mieux- -> Color by tag
On ne peut pas revenir au color by tag.
***
Pas de problème chez moi, ou je n'ai pas compris le problème.
***
J'avais mal précisé : cela fonctionne bien dans le scatterplot, mais pas dans la liste des clones.
***
Ah oui. Pour préciser : la couleur par tags est bien remise mais l'ancienne couleur (par J ou locus ou …) est conservée. On a donc un mélange de deux colorations.
***
>9a3f5e59949007
getColor() renvoyait un undefined quand aucune couleur n'était défini, étrangement on ne peut pas overwrite une valeur css avec undefined.
***
Super, merci Marc !
***
@magiraud @mikael-s @Duezhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2028Merger tous les types de sample_set, ou non ?2017-03-14T12:07:51+01:00Vidjil TeamMerger tous les types de sample_set, ou non ?http://dba.stackexchange.com/questions/109210/does-an-empty-column-value-occupy-same-storage-space-as-a-filled-column-value
En résumé, une colonne de type fixe occupera sa place qu'elle soit NULL ou non, alors qu'une colonne de type v...http://dba.stackexchange.com/questions/109210/does-an-empty-column-value-occupy-same-storage-space-as-a-filled-column-value
En résumé, une colonne de type fixe occupera sa place qu'elle soit NULL ou non, alors qu'une colonne de type variable n'occupera aucune place si elle est vide.
Il y a cependant un vecteur de bits en entête de la table, qui recense les colonnes NULLABLE, et occupe un bit par colonne (arrondi à l'octet supérieur).
Web2py créé des varchar pour les string et des longtext pour les text. Les longtext n'étant pas stockés dans la table à proprement dire, occupent toujours la place d'un pointeur.
En l'occurrence, le seul longtext que nous avons pour les sample_set correspond à la colonne info, commune à nos trois types de sample_set.
La seule colonne qui occuperait donc de la place inutile serait la date.
***
Donc l'enjeu serait plutôt de voir les performances de la liste. Si nous mettons tout en commun dans une table, il risque d'y avoir des performances similaires pour toutes les listes (patient, sample_set, run) quelque-soit le nombre de chaque type de sample_set.
Ce qui relève le sujet de la pagination.
***
Ceci dit, le nombre de sample_sets reste le même dans tous les cas.
***
merci Ryan pour ces éléments très précis !
***
@RyanHerbRyan HerbertRyan Herberthttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2027Beaucoup de FAILED dans le Vidjil Morning du jour2018-05-23T16:54:58+02:00Vidjil TeamBeaucoup de FAILED dans le Vidjil Morning du jourgrep -i failed
***
BIzarre, cette commande ne retourne rien :
grep -P '2016-11-08 1[45]' /var/vidjil/vidjil-debug.log
***
Je mets de côté les FAILED qui sont sur dev. Tous les autres FAILED concernent des fichiers de séquences supprimé...grep -i failed
***
BIzarre, cette commande ne retourne rien :
grep -P '2016-11-08 1[45]' /var/vidjil/vidjil-debug.log
***
Je mets de côté les FAILED qui sont sur dev. Tous les autres FAILED concernent des fichiers de séquences supprimés. Peut-être que les process étaient schédulés et l'utilisateur s'est rendu compte d'une erreur et a supprimé le patient. Du coup les process étaient ensuite en FAILED
Kiel a effectivement supprimé un patient à 15:22
2016-11-08 15:22:10,424 INFO - patient.py:432 194.94.189.101 <Markerscreening> patient (4085) deleted
Si je prends le run sch-236571, son last_run_time est 15:22:43 (j'en ai regardé d'autres et les horaires sont dans ces eaux-là aussi).
Tout colle :)
***
@magiraud @mikael-shttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2025Les smaller clones disparaissent de la liste dès qu'on les survole2016-11-29T14:43:23+01:00Vidjil TeamLes smaller clones disparaissent de la liste dès qu'on les survoleSuite à https://producteev.com/workspace/t/58204b9a2adaeada53000016 les smaller clones sont bien affichés de la liste mais disparaissent dès qu'on essaie de les survoler dans la liste.
***
Je précise qu'ils restent bien présents physique...Suite à https://producteev.com/workspace/t/58204b9a2adaeada53000016 les smaller clones sont bien affichés de la liste mais disparaissent dès qu'on essaie de les survoler dans la liste.
***
Je précise qu'ils restent bien présents physiquement dans la liste, mais sont alors en display: none
***
Merci pour ta vigilance.
6c623048
***
@nobodyhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2024Nommer plus clairement les groupes utilisateurs2020-07-01T11:12:47+02:00Vidjil TeamNommer plus clairement les groupes utilisateursOn pourrait changer "Group uniquely assigned to used 97" en "Group for user 0097 : Joe Doo", que ce soit à la création (et, si possible, rétroactivement, sur les groupes existants).
Cela permettrait de mieux comprendre, sur la page "grou...On pourrait changer "Group uniquely assigned to used 97" en "Group for user 0097 : Joe Doo", que ce soit à la création (et, si possible, rétroactivement, sur les groupes existants).
Cela permettrait de mieux comprendre, sur la page "groups", qui possède quel droits.
***
@RyanHerbhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2023Merge reads et .fasta2018-04-16T16:17:49+02:00Vidjil TeamMerge reads et .fastaÉchange entre Shun et Mikaël :
> The read merging failed because you provided FASTA files. When merging reads, you should provide FASTQ files only. We should specify that limitation more clearly. You cannot provide FASTA files if you wa...Échange entre Shun et Mikaël :
> The read merging failed because you provided FASTA files. When merging reads, you should provide FASTQ files only. We should specify that limitation more clearly. You cannot provide FASTA files if you want to merge the reads.
On devrait pouvoir utiliser le merge même avec des .fasta (quitte à faire un .fastq bidon). En attendant, j'ai renommé la config pre-process en "Merge paired-end reads (+R2) [only .fastq / .fastq.gz files]".
***
@magiraud @mikael-shttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2022Tester l'affichage des "smaller clones"2017-11-18T12:31:19+01:00Vidjil TeamTester l'affichage des "smaller clones"Ex ici : http://rbx.vidjil.org/browser/index.html?patient=4011&config=25
Plus de smaller clones dans la liste des clones. Où sont-ils passés ?
***
Cela vient de 2095ef3152, je ne sais pas encore pourquoi.
***
7525317c2
***
Ce serait bien...Ex ici : http://rbx.vidjil.org/browser/index.html?patient=4011&config=25
Plus de smaller clones dans la liste des clones. Où sont-ils passés ?
***
Cela vient de 2095ef3152, je ne sais pas encore pourquoi.
***
7525317c2
***
Ce serait bien de faire un test fonctionnel : vérifier qu'un clone virtuel "smaller clones" apparait dans la liste, mais pas dans le scatterplot.
(Cela dit, un jour on aura des clones "virtuels" qui iront aussi dans le scatterplot, ceux rajoutés manuellement...)
***
a11c031 (le commit montre la simplicité des tests Watir ;))
Cela va casser les tests à cause de https://producteev.com/workspace/t/58220195b0fa09ce7e000000
***
@nobody @magiraudhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2021Sauvegarde de l'analyse ne fonctionne pas2017-07-05T09:15:55+02:00Vidjil TeamSauvegarde de l'analyse ne fonctionne pasNotification mise ;)
Je suis dessus pour ~40min
***
Notification mise ;)
Je suis dessus pour ~40min
***
Sur dev.vidjil.org, j'ai modifié un self.m.samples.ids en self.m.samples.id dans databse.js, ligne 720 et ça fonctionne.
Mais ce n'es...Notification mise ;)
Je suis dessus pour ~40min
***
Notification mise ;)
Je suis dessus pour ~40min
***
Sur dev.vidjil.org, j'ai modifié un self.m.samples.ids en self.m.samples.id dans databse.js, ligne 720 et ça fonctionne.
Mais ce n'est pas une ligne modifiée récemment, donc ce n'est pas ça qui doit causer la nouveauté du problème.
Je n'arrive pas à trouver où ces ID sont définis dans le .js ni si quelquechose a récemment été modifié dans ce sens. Marc, Ryan une idée ?
Je ne suis plus dessus, vous avez la main ;)
***
Sur dev.vidjil.org, j'ai modifié un self.m.samples.ids en self.m.samples.id dans databse.js, ligne 720 et ça fonctionne.
Mais ce n'est pas une ligne modifiée récemment, donc ce n'est pas ça qui doit causer la nouveauté du problème.
Je n'arrive pas à trouver où ces ID sont définis dans le .js ni si quelquechose a récemment été modifié dans ce sens. Marc, Ryan une idée ?
Je ne suis plus dessus, vous avez la main ;)
***
C'est bien cette modification qui allait bien. Ou du moins c'était la plus simple. Le code de get_data (default.py) avec changé en octobre, donc quand j'ai mis prod-server à jour avec dev, le save_analysis. J'aurais pu remettre le bon champ "ids" dans le get_data mais il y avait plus de lignes à changer (au moins le double !).
***
@nobodyhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2020Clone majoritaire qui disparait avec un nombre plus important de samples2019-08-20T10:34:34+02:00Vidjil TeamClone majoritaire qui disparait avec un nombre plus important de samplesExemple : Dans le deuxième il y a un clone très clair en TRG qui disparait dans le premier cas
http://rbx.vidjil.org/browser/?custom=16705&custom=16717&custom=16729&custom=16741&custom=16753&custom=16762&custom=16771&custom=16780&
http:...Exemple : Dans le deuxième il y a un clone très clair en TRG qui disparait dans le premier cas
http://rbx.vidjil.org/browser/?custom=16705&custom=16717&custom=16729&custom=16741&custom=16753&custom=16762&custom=16771&custom=16780&
http://rbx.vidjil.org/browser/?custom=16705&custom=16717&custom=16729&custom=16741&
***
Cela semble dépendant de l'ordre. Essayé en local : selon les fichiers mis en premier les résultats obtenus ne sont pas les mêmes…
***
C'est ma faute : 0d40d04c7975403b691781b50eafef9efb1dced5
***
Je ne suis pas sûr de comprendre, merci en tout cas d'avoir trouvé.
Proposition : dans l'urgence, reverter simplement ce commit plutôt que de chercher à corriger. Puis changer une batterie de tests et changer le fonctionnement de fuse.py tranquillement.
***
Moi non plus je ne comprends pas.
***
Ah, c'est plus embêtant... Peux-tu mettre un test quelque part, même très informel, où on voit ce qui ne va pas ?
***
Commit réverté, utilisateur prévenu;
***
Merci ! Est-ce que cela ne vaudrait pas le coup de faire une notification sur le serveur pour tous ?
***
La mise à jour de prod-server sur rbx n'a été faite que hier et depuis il n'y a que Kiel qui a utilisé le fuse (ou lancé des jobs), du coup je ne pense pas que ce soit utile.
***
ok, parfait
***
@Duez @RyanHerb @mikael-s @magiraudhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2018L'alignement multiple ne se fait pas nécessairement par rapport à la première...2016-11-29T14:43:17+01:00Vidjil TeamL'alignement multiple ne se fait pas nécessairement par rapport à la première séquencecf. mail d'Aurélie 24/10 8h55
***
2998886
***
@mikael-scf. mail d'Aurélie 24/10 8h55
***
2998886
***
@mikael-shttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2016Charger des fichiers BAM2017-07-07T15:46:09+02:00Vidjil TeamCharger des fichiers BAMLes fichiers BAM sont directement générés par les séquenceurs Ion apparemment, cela éviterait de passer par le FASTQ. Le fichier BAM est une version compressée du fichier SAM. Il vaut mieux passer par une bibliothèque externe pour les li...Les fichiers BAM sont directement générés par les séquenceurs Ion apparemment, cela éviterait de passer par le FASTQ. Le fichier BAM est une version compressée du fichier SAM. Il vaut mieux passer par une bibliothèque externe pour les lire… Il existe par exemple htslib mais elle fait plein d'autres choses https://github.com/samtools/htslib (il y en a probablement d'autres)
***
@magiraud @mikael-sAlgo 2017.07Mikaël SalsonMikaël Salsonhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2014Le merge semble ne pas fonctionner2016-11-29T14:43:14+01:00Vidjil TeamLe merge semble ne pas fonctionnerEn réalité il fonctionne, mais les bulles ne disparaissent pas et la liste affiche un – alors que le clone est « replié », ce qui donne l'impression que le merge ne fonctionne pas.
***
75e01801 c'était pour réparer les tests fonctionnels...En réalité il fonctionne, mais les bulles ne disparaissent pas et la liste affiche un – alors que le clone est « replié », ce qui donne l'impression que le merge ne fonctionne pas.
***
75e01801 c'était pour réparer les tests fonctionnels.
***
Oui mais ça ne le corrige pas dans le bon sens ;) Le clic sur le bouton merge ne provoque pas une « fusion » des bulles, ce qui est un peu contre-intuitif.
***
Le commit mentionné était à l'origine du problème, je l'ai revert, donc le merge fonctionne à nouveau, mais le test qui était réparé par le commit est de nouveau en erreur.
***
merci, super !
***
@RyanHerb @Duezhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2013Un mauvais commit traine dans dev, à rebaser2017-02-01T18:48:35+01:00Vidjil TeamUn mauvais commit traine dans dev, à rebaser1bba34fa35 (XXXX choses qui trainaient dans ~/git/prod-server/ XXX) s'est (encore) retrouvé sur dev :-)
***
@magiraud @RyanHerb @mikael-s1bba34fa35 (XXXX choses qui trainaient dans ~/git/prod-server/ XXX) s'est (encore) retrouvé sur dev :-)
***
@magiraud @RyanHerb @mikael-shttps://gitlab.inria.fr/vidjil/vidjil/-/issues/2012should-to-tap.sh / should-vdi-to-tap.py : option verbose, -v/-vv ?2022-06-20T18:12:36+02:00Vidjil Teamshould-to-tap.sh / should-vdi-to-tap.py : option verbose, -v/-vv ?Il m'arrive très souvent, pour enquêter sur un test, d'aller voir le .log pour retrouver la commande vidjil et la lancer, éventuellement modifiée.
Avec should-to-tap.sh, on voit la commande lancée. Ce serait bien d'avoir une option dans...Il m'arrive très souvent, pour enquêter sur un test, d'aller voir le .log pour retrouver la commande vidjil et la lancer, éventuellement modifiée.
Avec should-to-tap.sh, on voit la commande lancée. Ce serait bien d'avoir une option dans should-vdj-to-tap.py pour voir cela ou de le mettre par défaut (mais encombre la sortie)
Et ce serait bien, pour les deux, d'avoir une option pour voir directement le log. Je pinaille, peut-être est-ce du confort juste pour moi.
***
@magiraud @mikael-s