Detobel
Detobel
Tested for function `get_decision_frontier` here https://github.com/NVISO-BE/ee-outliers/blob/master/app/tests/unit_tests/test_utils.py#L303 Test "in real" conditions here: https://github.com/detobel36/ee-outliers/blob/generatorDoc/app/tests/unit_tests/test_analyzer_metrics.py#L829 and here: https://github.com/detobel36/ee-outliers/blob/generatorDoc/app/tests/unit_tests/test_analyzer_terms.py#L264
Since #308 there is no more order to test the "literal" whitelist value. Complicated for regex, because `dict_values_to_check` is a set (no order) to remove duplicate information. To remove that...
Wait wait, y a pas une norme Kodi (basé sur des regex) ? https://kodi.wiki/view/Scrapers Personnellement cette idée m'intéresse beaucoup. Car elle permettrait (comme proposer) de "séparer" le problème en deux:...
Je suis d'accord que ça va prendre du temps et que c'est plus une "tâche de fond". Cependant, je pense qu'il pourrait être intéressant de déjà faire en sorte que...
PEP c'est la norme :smile: Mais on est "libre" de modifier nos règles si on "aime pas une règle" :wink: Pour "justifier" cette proposition (et pas juste dire "faut suivre...
> Peux-tu nous donner u lien vers une version light de la norme ? C'est à nous de la définir la version "light" :wink: En gros PEP8 c'est la norme....
Petite question en lien avec ça... Est-ce que les autres addon (on a évoqué Track, TMDBHelper, ...) prennent en compte le fait qu'on ai vu un film via vStream ?...
> Tu parles des tests automatisé de github ? Pour lancer les tests, oui je pensais aux "actions" github > Le sites scrapes etant dynamique, un test ne voudrait pas...
Je suis tombé la dessus: https://github.com/Catch-up-TV-and-More/plugin.video.catchuptvandmore/blob/dev/.github/workflows/ci.yml A voir si on sait "faire la même chose" et si ça "vaut la peine" :wink:
Pour être sûr à 100% d'avoir tous les résultats, il faudrait lancer plusieurs recherches. Par exemple: > Le crime de l'orient-express (titre TMDB) On pourrait faire deux recherches: - `le%20crime%20de%20l%27orient-express`...