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

Add new systems with electrolyzers #4

Merged
merged 2 commits into from
Feb 13, 2024
Merged

Conversation

tbittar
Copy link
Contributor

@tbittar tbittar commented Feb 12, 2024

No description provided.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne vois pas (ou je n'ai pas compris^^) dans le modèle le fait que "that minimum production could be curtailed at a defined cost"

Copy link
Contributor

@Juliette-Gerbaux Juliette-Gerbaux Feb 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est le p qui permet de contrôler le niveau d'effacement, s'il est égal à p_max, il n'y en a pas et c'est gratuit et on peut avoir de l'effacement d'au maximum p_max-p_min

Copy link
Contributor Author

@tbittar tbittar Feb 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour l'instant on s'interdit dans le code d'écrire dans les modèles des expressions avec des ports dans les seconds membres.
J'aurais plutôt mis les "flèches" dans l'autre sens (de l'electrolyseur et de la prod ENR vers le contrat). Et dans le modèle Contract

Prod_electrolyzers, prod_enr : Variable
contracted_production : Parameter
flow_d.f.sum() = prod_electrolyzers
flow_p.f.sum() = prod_enr
prod_electrolyzers = min(prod_enr, contracted_production)

On crée peut-être des variables intermédiaires inutiles à cause des équations de flow.f.sum()
Et ce que je comprends, c'est que les 2 autres onglets décrivent le même fonctionnement mais en aggrégeant plus ou moins les choses dans un même modèle ou pas

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai modifié un peu différemment de ce que tu as proposé pour ne créer qu'une seule variable.
C'est bien ça pour les autres onglets, je me suis dit que c'était plus clair si je mettais les différentes options dans différents onglets plutôt que sur le même dessin

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok ok j'ai l'impression que ça marche mais là on voit bien l'intérêt d'écrire explicitement les équations qu'on veut représenter parce que sinon c'est assez difficile de s'assurer qu'on représente bien ce qu'on veut

@tbittar
Copy link
Contributor Author

tbittar commented Feb 12, 2024

Il faudrait uniformiser entre electrloyser et electrolyzer dans le nom de certains fichiers

@Juliette-Gerbaux Juliette-Gerbaux merged commit c0661d9 into main Feb 13, 2024
1 check passed
@Juliette-Gerbaux Juliette-Gerbaux deleted the models/electrolyzer branch February 21, 2024 09:23
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