JP code review

[jpmjpmjpm]

Sur le code

Pour lire un fichier ligne à ligne, il est intéressant d'utiliser with. Cela évite d'avoir à fermer le fichier manuellement. Laisser trop de fichiers ouverts risque de créer une erreur système... pas de problème pour peu de fichiers cependant!

Pour créer une liste avec n valeurs identiques, i.e. 0: [0] * n.

Éviter d'utiliser une variable globale dans une fonction, comme par exemple data ici:

def optimal(dt):
    size = len(data)

J'ai créé une classe ici pour pouvoir gérer plus facilement les données. Aussi, un exemple dans le fichier d'implémentation de stratégies sous forme de fonction, qui sont ensuite appelées par une fonction générique qui effectue le calcul des regrets. Pour éviter d'avoir à reproduire la logique de calcul à chaque fois.

Sur le projet Data Science

• Tu utilises bien les fonctions numpy! Tu pourrais utiliser plus souvent numpy.sum pour éviter de faire quelques boucles à la main.

Quelques questions / commentaires

• Quand tu rends un projet, tout le code doit-il être dans un notebook? Ou peux-tu avoir des scripts python séparés que tu importes? Je pose la question car c'est plus simple de créer des classes quand tu peux avoir des scripts séparés.

• Tu n'utilises pas pandas. Est-ce volontaire?

• Je n'ai pas regardé si le code réalise bien les algorithmes demandés... ce qui serait peut-être le plus intéressant!