vidjil issueshttps://gitlab.inria.fr/vidjil/vidjil/-/issues2018-06-13T15:45:33+02:00https://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ée