À mon avis, ça ne vaut pas la peine d'ajouter des constantes pour les noms qui se trouvent dans la config. Il y aura une erreur si le nom n'existe pas dans le dictionaire de config de toute façon. On aurait juste des constantes du genre:
FUEL_CELL_A = "FUEL_CELL_A"
À mon avis, ça fait beauoup de répétition pour pas grand chose. J'ai l'impression que c'est relativement "standard" d'avoir des dictionaires comme ça dans python et les indexer avec des clés plutôt que des types comme des structs qu'on pourrait voir dans d'autre languages.
En tout cas, j'ai créé une issue ici pour en discuter, surtout que ça affecte plus que le code qui a été modifié dans le PR.
Follow-up de https://github.com/Exocet-Polytechnique/ExoPIBrain/pull/12#discussion_r1482286573
À mon avis, ça ne vaut pas la peine d'ajouter des constantes pour les noms qui se trouvent dans la config. Il y aura une erreur si le nom n'existe pas dans le dictionaire de config de toute façon. On aurait juste des constantes du genre:
À mon avis, ça fait beauoup de répétition pour pas grand chose. J'ai l'impression que c'est relativement "standard" d'avoir des dictionaires comme ça dans python et les indexer avec des clés plutôt que des types comme des
structsqu'on pourrait voir dans d'autre languages.En tout cas, j'ai créé une issue ici pour en discuter, surtout que ça affecte plus que le code qui a été modifié dans le PR.