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

fix(thermals): correct the default value of the "gen-ts" property to use "use global" instead of "use global parameter" #1918

Merged
merged 2 commits into from
Jan 31, 2024

Conversation

laurent-laporte-pro
Copy link
Contributor

@laurent-laporte-pro laurent-laporte-pro commented Jan 30, 2024

Correction

This PR corrects the default value of the "gen-ts" property to use "use global" instead of "use global parameter"

The enumerated type TimeSeriesGenerationOption is corrected to take into account the value use global (instead
of use global parameter).

Other refactoring

  • The TimeSeriesGenerationOption class has been renamed to LocalTSGenerationBehavior to match the Antares Solver
    nomenclature and simplify maintenance.

Tests

  • Tests have been updated to check the correct behavior of the LocalTSGenerationBehavior class.
  • Integration tests have been done to check the dropdown list of values in the thermal cluster form.

Screenshot

image

Closes #1917

…the configuration of thermal clusters

The enumerated type `TimeSeriesGenerationOption` is corrected to take into account the value `use global` (instead of `use global parameter`).
…enamed to `LocalTSGenerationBehavior` to match the Antares Solver nomenclature
@laurent-laporte-pro laurent-laporte-pro merged commit 37594a2 into dev Jan 31, 2024
6 checks passed
@laurent-laporte-pro laurent-laporte-pro deleted the bugfix/1917-correct-gen-ts-vaue branch January 31, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Invalid value for thermal param "gen-ts"
2 participants