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

Merge branch release/8.8.x into develop (changelog, version number) #1833

Merged
merged 11 commits into from
Dec 21, 2023

Conversation

flomnes
Copy link
Member

@flomnes flomnes commented Dec 19, 2023

Updated changelog, version number

flomnes and others added 7 commits November 21, 2023 16:35
…ing ST objects (#1807)

- Add bool property `enabled` for each ST storage object
- Whenever a ST storage is not `enabled`, remove it from computations
relatively early (right after loading the data), using a similar
mechanism as thermal & renewable clusters
- Also fix list.ini files containing only the last ST storage object
because of overwrite, add unit test case
- Cherry-pick of "Remove sts container (#1804)"

---------

Co-authored-by: Vincent Payet <[email protected]>
Co-authored-by: payetvin <[email protected]>
Copy link

watermelon-copilot-for-code-review bot commented Dec 19, 2023

Watermelon AI Summary

AI Summary deactivated by flomnes

GitHub PRs

Antares_Simulator is an open repo and Watermelon will serve it for free.
🍉🫶

@flomnes flomnes changed the title Release/8.8.x Merge branch release/8.8.x into develop Dec 19, 2023
@@ -278,14 +278,16 @@ class STstorageCashFlowByCluster : public Variable::IVariable<STstorageCashFlowB
const auto& shortTermStorage = results.data.area->shortTermStorage;

// Write the data for the current year
for (uint clusterIndex = 0; clusterIndex < nbClusters_; ++clusterIndex)
uint clusterIndex = 0;
Copy link
Member Author

Choose a reason for hiding this comment

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

clusterIndex appears to be unused

src/solver/variable/economy/STStorageInjectionByCluster.h Outdated Show resolved Hide resolved
src/solver/variable/economy/STStorageLevelsByCluster.h Outdated Show resolved Hide resolved
src/solver/variable/economy/STStorageWithdrawalByCluster.h Outdated Show resolved Hide resolved
docs/CHANGELOG.md Outdated Show resolved Hide resolved
@pull-request-size pull-request-size bot added size/M and removed size/L labels Dec 21, 2023
@flomnes flomnes changed the title Merge branch release/8.8.x into develop Merge branch release/8.8.x into develop (changelog, version number) Dec 21, 2023
Co-authored-by: abdoulbari zakir <[email protected]>
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@flomnes flomnes merged commit e4f486d into develop Dec 21, 2023
8 checks passed
@flomnes flomnes deleted the release/8.8.x branch December 21, 2023 12:20
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.

2 participants