Skip to content

Add reserve model#6

Merged
tbittar merged 8 commits intomainfrom
models/reserve
Mar 8, 2024
Merged

Add reserve model#6
tbittar merged 8 commits intomainfrom
models/reserve

Conversation

@tbittar
Copy link
Collaborator

@tbittar tbittar commented Feb 12, 2024

No description provided.

@Juliette-Gerbaux
Copy link
Contributor

J'ai fait quelques modifications pour que ce soit tout en anglais avec des majuscules et des minuscules cohérentes.

@tbittar
Copy link
Collaborator Author

tbittar commented Feb 22, 2024

  • Il faudrait ajouter les équations mathématiques du modèle de réserve que tu veux représenter dans le dessin (cf. les modèles de Juliette)
  • Le modèle demand est dupliqué, supprimer une instance
  • Essayer d'utiliser une syntaxe pour visualiser rapidement ce qui est dupliqué et allegé les modèles (nombre de champs dans les ports de réserve, etc..), par exemple en utilisant r_{i}_up et r_{i}_down
  • Pourquoi c1_exp_up dans le modèle LinkWithReserve plutôt que r1_to_up, le c a-t-il une signification ? Et pour comprendre l'équation de contrainte des flux je trouve plus clair d'utiliser from et to comme ailleurs dans les modèles, à moins qu 'il y a une autre raison pour exp et imp ?
  • res_to.r2_down = - (c2_exp_down - c2_imp_down) je pense que c'est l'opposé
  • Faire attention à bien uniformiser le nom des ports dans la description des modèles et le dessin (ex : dans le modèle Thermal, port res mais res_r sur le dessin, dans le ReserveConverter, res_from et res_to mais res_out et res_in sur le dessin)
  • Par cohérence, le nom des modèles doit être tout attaché avec les premières lettres des "mots" en majuscule : Reserve converter -> ReserveConverter, à appliquer sur tous les modèles
  • Dans le modèle ReserveConverter, les variables r{i}_up_down ne sont pas déclarées, et je ne comprends pas trop la convention de nommage entre r1_up_c et r1_up_down, que signifie le c ? Pourquoi n'y a-t-il pas de symmétrie (dans les noms) entre les cas up et down ?
  • Que signifient les bordures oranges autour de certains ports / composants ?

@thibaulttoujouse
Copy link
Contributor

  • J'ai défini la syntaxe r_{i}up et r{i}_down pour visualiser ce qui est dupliqué. Si on opte à terme pour une représentation en liste pour les réserves, une mise à jour sera possible

  • Dans LinkWithReserve, on définit des variables "c1_exp_up "... car cela représente une réservation de capacité, d'où le "c". On définit bien dans le modèle "from" et "to" dans les ports de sortie. On reste tout de même obligé de définir ce qui est des imports hausse/baisse et exports hausse/baisse dans les variables car selon si on importe ou exporte, la variable ne joue pas le même rôle dans la contrainte.

-Dans le modèle "ReserveConverter", les notations ont bien été modifiées (erreur de copier/coller).

Les autres remarques ont bien été prises en compte.

@Juliette-Gerbaux
Copy link
Contributor

Le texte de présentation du modèle est en français, ça serait bien de le traduire en anglais pour être plus cohérent

@tbittar tbittar merged commit 83ca0d8 into main Mar 8, 2024
@tbittar tbittar deleted the models/reserve branch March 8, 2024 16:38
aoustry added a commit that referenced this pull request May 13, 2025
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.

3 participants