WIP: Code Review sur kraken_benchmark.py

Cette code review ne porte que sur kraken_benchmark.py.

Voir les logs pour la description des modifications, à partir du commit b37b90d3af2373bd5b3e4268fa3818d744040c99:

commit 9c301e293d2e77b4d694764c22362a36c1efd8c0 (HEAD -> kb-app_review, origin/kb-app_review, KB-app)
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 15:44:36 2020 +0200

    simplification des structures de get_transcription()
    en inversant le if/else, tu te libères d'un niveau d'imbrication puisqu'en cas d'erreur, le programme est interrompu avant de passer à la suite.
    voir les commentaires 'Review' pour des remarques ponctuelles sur des points à améliorer.

⏫

commit f7c1edac0f3d898cb67e094e052c18f3cc993376
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 15:29:37 2020 +0200

    creation d'une function generate_html_report()
    noter que la génération de l'id peut se faire dans la fonction également. Ce genre de bloc d'exécution doit faire l'objet d'une fonction à part entière.
    La dosctring est à compléter.

⏫

commit e0646375e13b1745234cb9cc07ecb66822bbf92e
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 15:04:12 2020 +0200

    ajout d'un argument input
    qui permet plus de flexibilité sur la localisation des fichiers sources. Si spécifiée, la valeur d'input n'est pas prise en compte
    pour le moment, puisqu'il faut d'abod adapter load_input(). Rétablissement des versions longues et courtes des arguments pour plus
    de lisibilité.

⏫

commit 627d0280312d8ccfe05dbaeef64df5f2a8aacbe9
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 14:41:10 2020 +0200

    optimisation de l'utilisation de try/except
    le try est à l'intérieur de la boucle : cela permet d'identifier plus vite quelle image a posé problème, et évite
    d'utiliser try/except sur toute la boucle puisqu'en général on préfère limiter le nombre d'instructions dans un
    bloc try. Au mieux 1 try = 1 instruction.
    Par ailleurs, déplacement de la création des listes vides après le commentaire signalant le début d'une nouvelle
    étape.

⏫

commit bd27ab3e2954f9ab6a94cc61a95f6e95e96d4105
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 14:34:14 2020 +0200

    ignore correction sur pbar de tqdm

⏫

commit bccb9d32c89a7bc44810533dcb6b1944f916d491
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 14:22:34 2020 +0200

    petite correction sur l'utilisation de tqdm

⏫

commit a8cdeeb58e0c136854fe37e575eeef204bd64689
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 12:25:59 2020 +0200

    crédit

⏫

commit f16abfe1991aa9ab12c046b2a0423194fb8c6b39
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 12:25:12 2020 +0200

    ajout d'une fonction de rapport et implémentation
    avec la fonction report_log(message, type), tu simplifies la commannde pour afficher les rapports
    j'ai aussi mis les éléments relevant de l'option verbose en bleu, mais ce n'est pas nécessaire

⏫

commit b5958a0758c40063d79828987486fbed491c4a01
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 11:58:15 2020 +0200

    refactorisation de result_logo
    le nom n'est pas explicite : kraken_benchmark_logo (ou logo tout simplement) - c'est un exemple et la correction ici est superflue

⏫

commit c7fe5c35fea26615a5ebec01a53ba26dc876c8cb
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 11:55:20 2020 +0200

    refactorisation des constantes
    RESULT_LOGO, ARGS et PARSER ne sont pas des constantes, donc passage du nom en minuscule

⏫

commit 2f2bf99fbd6436bfe5ff9b549de6cfb1265a365e
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 11:51:55 2020 +0200

    arrengement des imports
    la règle est : un bloc pour les imports de libs built-in, un bloc pour les imports de lib externes,
    un bloc pour les imports de lib et modules locaux. Ensuite, à l'intérieur de ces blocs, les imports
    sont rangés dans l'ordre alphabétique. Pas de saut de ligne à l'intérieur des imports.

⏫

commit 3e523b66771ff98aa7791d57b70241bb198e15f6
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 11:49:25 2020 +0200

    retrait des sauts de lignes inutiles
    chacun sa pratique, cependant un code trop aéré n'est pas toujours très lisible. Il faut garder
    les sauts de lignes pour distinguer des unités logiques. Donc ne pas sauter de ligne après un
    commentaire qui se rapporte à la ligne de code suivante, par exemple.

⏫

commit b37b90d3af2373bd5b3e4268fa3818d744040c99
Author: Alix Chagué <alix.chague@inria.fr>
Date:   Mon Jun 22 11:32:04 2020 +0200

    dummy_màj requirements