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

feat(hydro): add "Min Gen." tab to Hydro section for studies in v8.6 and later #1932

Merged
merged 3 commits into from
Feb 13, 2024

Conversation

laurent-laporte-pro
Copy link
Contributor

This new feature involves adding a "Min Gen." tab to the Hydro section. This tab is specifically for studies in version 8.6 or later. The "Min Gen." tab is dedicated to configuring the minimum generation levels of the hydraulic generators. It presents a time series that represents the minimum hourly production for one or more Monte-Carlo years.

The implementation includes:

  • the addition of a new route "mingen" at the same level as other Hydro routes,
  • the addition of a new matrix type "MinGen" in HydroMatrixType,
  • the addition of a screenshot in the documentation with a brief description.

Testing:

  • For studies in version 8.6 or later, the "Min Gen." tab should be displayed and the matrix should be loaded.
  • For studies in versions earlier than 8.6, the "Min Gen." tab should not be displayed.

@laurent-laporte-pro laurent-laporte-pro self-assigned this Feb 13, 2024
@laurent-laporte-pro laurent-laporte-pro force-pushed the feature/add-mingen-tab-frontend branch from 2826b6b to 6a2c4c5 Compare February 13, 2024 13:25
@laurent-laporte-pro laurent-laporte-pro changed the title feat(hydro): add "Min Gen." Tab to Hydro section for studies in v8.6 and Later feat(hydro): add "Min Gen." tab to Hydro section for studies in v8.6 and later Feb 13, 2024
@skamril skamril force-pushed the feature/add-mingen-tab-frontend branch from 6a2c4c5 to 43bf505 Compare February 13, 2024 13:52
@@ -30,7 +31,8 @@ function Hydro() {
{ label: "Water values", path: `${basePath}/watervalues` },
{ label: "Hydro Storage", path: `${basePath}/hydrostorage` },
{ label: "Run of river", path: `${basePath}/ror` },
];
studyVersion >= 860 && { label: "Min Gen.", path: `${basePath}/mingen` },
].filter(Boolean);
}, [areaId, study?.id]);
Copy link
Member

Choose a reason for hiding this comment

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

missing dependency studyVersion here

@@ -144,6 +149,11 @@ export const MATRICES: Matrices = {
url: "input/hydro/series/{areaId}/ror",
stats: MatrixStats.STATS,
},
[HydroMatrixType.MinGen]: {
title: "Min Gen.",
Copy link
Member

Choose a reason for hiding this comment

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

Could you please remove all dots "." from tabs label, matrix title etc

@hdinia hdinia force-pushed the feature/add-mingen-tab-frontend branch from 8a8dccb to e576108 Compare February 13, 2024 17:36
@laurent-laporte-pro laurent-laporte-pro merged commit a6e9bb6 into dev Feb 13, 2024
7 checks passed
@laurent-laporte-pro laurent-laporte-pro deleted the feature/add-mingen-tab-frontend branch February 13, 2024 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants