Grid2op / chronix2grid

https://chronix2grid.readthedocs.io/en/latest/?
Mozilla Public License 2.0
19 stars 5 forks source link

Lots of fixes, some addition #69

Closed BDonnot closed 1 year ago

BDonnot commented 1 year ago
marota commented 1 year ago

Merci très bonne PR. Quelques commentaires non dimensionnants: proposition de mettre à jour un paramètre de doc là ou la doc existe déjà, 3 propositions de passer des blocs de code dand une fonction/méthode, une question sur l'impact des maintenances éoliennes sur les forecasts.

Avant de prochaines evols, il faudra sans doute faire une pass de doc et de tests sur grid2op_utils pour ne pas perdre le fil des développements. Et rajouter une sous-section "forecast model" dans la section "models" de la doc (sur la base d'un document déjà existant)

BDonnot commented 1 year ago

On pourra faire effectivement une revue du code chronix2grid en bonne et due forme à un moment je suis d'accord. En l'état ce code est loin d'être "aux standards" (de grid2op / lightim2grid par ex). Ca serait bon, je pense de prendre quelques mois et de remettre tout ça au propre, maintenant qu'on connait un peu plus ce que ça doit faire et qu'on peut tester "d'un point de vue utilisateurs" ce qu'on veut.

BDonnot commented 1 year ago

Je te laisse merger quand les tests sur la modif que j'ai faite seront passés (ou pas) mais comme c'est de la doc ça devrait marcher.