Skip to content

Var env issue11

Benoît Courty requested to merge var_env_issue11 into master

Created by: magemax

Création de 3 super variables d'environnements pour répondre à l'issue 11 :

  • YEAR_COMPUTATION : année de calcul
  • EMPIRIC_VALUE : valeur du résultat attendu pour l'état
  • PLF_PATH : chemin où un éventuel PLF se trouve.

Questions résiduelles :

  • j'ai pas fait les cas types. Il faudra une bonne sélection d'emojis pour me convaincre qu'ils ont leur place dans cette PR.
  • Mes méthodes de "fail gracefully et fast" sont noramment présente dans l'implémentation du PLF_PATH. Pour l'instant j'utilise la technique dite du print & raise, qui est pas très classe. Comment logger de manière plus classe ?
  • Oui, je fais un exec sur une chaîne de caractères inconnue, mais je la regex un peu avant pour pas que le code exécute complètement n'importe quoi
  • les noms de variable sont pour l'instant moches : j'ai pas changé DATA_PATH et j'ai rajouté PLF_PATH. Des suggestions ? Les autres noms sont en anglais par cohérence.

Reste à faire :

  • implémenter checks sur les autres variables : fail si le data_path est faux ou si le base_results est vide. Ca dépendra des retours sur les techniques pour fail fast

@sandcha pas besoin à ce stade de review le code, les "questions résiduelles" portent sur la definition of done. Tu auras tout au plus besoin de jeter un coup d'oeil pour comprendre toutes les questions résiduelles.

Merge request reports

Loading