vidjil issueshttps://gitlab.inria.fr/vidjil/vidjil/-/issues2018-06-13T11:51:58+02:00https://gitlab.inria.fr/vidjil/vidjil/-/issues/3272Merge du travail de Cyprien et historique du git2018-06-13T11:51:58+02:00Mathieu GiraudMerge du travail de Cyprien et historique du git(Mis volontairement dans une issue et non pas une MR, car les branches vont changer)
Dans la perspective d'un merge prochain du travail de @boreec, j'ai regardé systématiquement les différences entre `dev` et `feature-a/920-optimize-fin...(Mis volontairement dans une issue et non pas une MR, car les branches vont changer)
Dans la perspective d'un merge prochain du travail de @boreec, j'ai regardé systématiquement les différences entre `dev` et `feature-a/920-optimize-fine-segmenter` sur les "anciens" fichiers (ce qui au passage a levé #3270, #3271).
Je me demande maintenant si on ne devrait pas simplifier l'historique pour le futur.
- Ce qui me gène le plus est a7d99993 qui touche à `align_against_collection`... alors qu'on final il n'y a absolument rien de différent dans `align_against_collection`. Je tiens à rebaser / enlever cela (je peux m'en charger).
- Plus généralement, les premières semaines il y a eu beaucoup de déplacements dans tous les sens, des fonctions essayées puis supprimées (c'est normal @boreec, cela nous arrive aussi quand on fait des choses de cet ampleur), et c'était le début et les commits n'étaient pas toujours très atomiques. Bref je me demande si on pourrait squasher (et faire quelques commits sur les différents fichiers) jusqu'à par exemple fin avril. Je peux également m'en charger si ceal vous convient.merge-filterMathieu GiraudMathieu Giraudhttps://gitlab.inria.fr/vidjil/vidjil/-/issues/3270Utiliser NO_LIMIT_VALUE plutôt que UINT_MAX dans filter.h/filter.cpp2018-06-13T15:45:33+02:00Mathieu GiraudUtiliser NO_LIMIT_VALUE plutôt que UINT_MAX dans filter.h/filter.cppUn détail : nous n'avons pas l'habitude d'utiliser des comparaisons avec `UINT_MAX`, mais plutôt de faire un test explicite avec `NO_LIMIT_VALUE` (voir les occurrences de `NO_LIMIT_VALUE` dans `core/*.cpp`).
(Cela simplifiera en plus #3...Un détail : nous n'avons pas l'habitude d'utiliser des comparaisons avec `UINT_MAX`, mais plutôt de faire un test explicite avec `NO_LIMIT_VALUE` (voir les occurrences de `NO_LIMIT_VALUE` dans `core/*.cpp`).
(Cela simplifiera en plus #3264.)merge-filterCyprien BoréeCyprien Boréehttps://gitlab.inria.fr/vidjil/vidjil/-/issues/3268-Z all : ne pas construire l'automate2018-06-13T15:45:33+02:00Mathieu Giraud-Z all : ne pas construire l'automateLorsqu'on désactive l'automate, on ne devrait pas construire l'automate.
Ce n'est pas le cas pour l'instant pour les résultats de #3260 (mais c'était intéressant pour voir le temps de construction de l'automate). En temps normal, si on n...Lorsqu'on désactive l'automate, on ne devrait pas construire l'automate.
Ce n'est pas le cas pour l'instant pour les résultats de #3260 (mais c'était intéressant pour voir le temps de construction de l'automate). En temps normal, si on ne veut pas d'heuristique, on ne veut pas prendre du temps / de la mémoire avec l'automate.merge-filterCyprien BoréeCyprien Boréehttps://gitlab.inria.fr/vidjil/vidjil/-/issues/3264Option -Z pour le "N" du filtrage par automate2018-06-13T15:45:34+02:00Mathieu GiraudOption -Z pour le "N" du filtrage par automateFaire une option `-Z nb` (en ce moment on a peu d'options dispos, ping #2732) pour spécifier en ligne de commande le N à utiliser. Cela demandera sûrement de passer cette valeur à travers quelques fonctions.
Dans l'optique d'un merge pr...Faire une option `-Z nb` (en ce moment on a peu d'options dispos, ping #2732) pour spécifier en ligne de commande le N à utiliser. Cela demandera sûrement de passer cette valeur à travers quelques fonctions.
Dans l'optique d'un merge prochain, la valeur par défaut sera pour l'instant `-Z all` (voir `NO_LIMIT` / `NO_LIMIT_VALUE` dans `vidjil.cpp`).merge-filterCyprien BoréeCyprien Boréehttps://gitlab.inria.fr/vidjil/vidjil/-/issues/3271Ne pas relancer filterBioReaderWithACAutomaton() dans vidjil.cpp2018-06-13T16:13:07+02:00Mathieu GiraudNe pas relancer filterBioReaderWithACAutomaton() dans vidjil.cppPour la sortie `clone_file_name` (ping #3269), e01870a (puis fe6f5c6, 9e26c74) ré-utilisent `filterBioReaderWithACAutomaton()` au moment de la *sortie* du clone déjà segmenté.
C'est dommage, on appelle au total `filterBioReaderWithACAut...Pour la sortie `clone_file_name` (ping #3269), e01870a (puis fe6f5c6, 9e26c74) ré-utilisent `filterBioReaderWithACAutomaton()` au moment de la *sortie* du clone déjà segmenté.
C'est dommage, on appelle au total `filterBioReaderWithACAutomaton()`... deux fois.
Pour enlever cela,
- soit stocker `filtered` dans le `FineSegmenter` (le plus simple ?)
- soit stocker directement `filtered.read(box_V->ref_nb)` dans `box_V`
cc @boreecmerge-filter