Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bd dev #3

Merged
merged 16 commits into from
Oct 6, 2023
Merged

Bd dev #3

merged 16 commits into from
Oct 6, 2023

Conversation

BDonnot
Copy link
Collaborator

@BDonnot BDonnot commented Sep 22, 2023

  • refactor slightly the exsisting code
  • add an easier way to use the extended ST
  • add a notebook (init) to compute the security analysis with lightsim2grid and pandapower

@BDonnot BDonnot requested a review from marota September 26, 2023 16:16
@marota
Copy link
Owner

marota commented Oct 4, 2023

@BDonnot est-ce que le fichier test.py à la racine est encore nécessaire ? J'ai l'impression qu'il y a un fichier de test plus récent dans tests/test_state.py qui doit le reprendre et faire plus non ?

@marota
Copy link
Owner

marota commented Oct 4, 2023

grid_state.py est le fichier principal avec la classe State et compute_flows de défini. Mais sachant qu'il y a un dossier State, on s'attendrait aussi à le trouver là-dedans. Comment gérer cette ambigüité ?

  1. Bouger le fichier dans dossier state ?
  2. renommer le dossier state ?
  3. renommer le fichier grid_state ?

@marota
Copy link
Owner

marota commented Oct 4, 2023

For after this PR, I created two issues to improve documentation #4 and make the compute flow works when combining multi-types of actions with State API #5

@marota
Copy link
Owner

marota commented Oct 4, 2023

Beside that, it looks good to me and huge thanks @BDonnot for improving some existing code, helping run experiments and proposing a more flexible code architecture for the future with the state API

@BDonnot
Copy link
Collaborator Author

BDonnot commented Oct 5, 2023

est-ce que le fichier test.py à la racine est encore nécessaire ? J'ai l'impression qu'il y a un fichier de test plus récent dans tests/test_state.py qui doit le reprendre et faire plus non ?

Oui en effet c'était des "tests" lorsque je développais les fonctionnalités et non des "tests unitaires" qui sont bien dans le bon repertoire.

@BDonnot
Copy link
Collaborator Author

BDonnot commented Oct 5, 2023

grid_state.py est le fichier principal avec la classe State et compute_flows de défini. Mais sachant qu'il y a un dossier State, on s'attendrait aussi à le trouver là-dedans. Comment gérer cette ambigüité ?

Oui en effet l'architecture actuelle n'est pas complète et il manque encore pas mal de choses sur lesquelles je voudrais travailler notamment:

  • mettre tout ça en backend
  • pouvoir lire un réseau pandapower
  • pouvoir lire un réseau pypowsybl

J'ai mis ce dossier "à la racine" parce que je ne suis pas encore fixé sur la façon d'organisé tout ça "a terme". Mais totalement d'accord sur le fait que ce ne soit pas optimal dans l'état actuel.

@marota marota merged commit 52179d5 into master Oct 6, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants