Skip to content

bugfix total installed PV/Wind/Batteries#131

Merged
AteZenmo merged 3 commits intomainfrom
total_PV_Installed_Bug
Mar 21, 2025
Merged

bugfix total installed PV/Wind/Batteries#131
AteZenmo merged 3 commits intomainfrom
total_PV_Installed_Bug

Conversation

@Luc-Sol
Copy link
Contributor

@Luc-Sol Luc-Sol commented Mar 20, 2025

  • The totals installed for PV / Wind and Batteries were not reset before being recalculated inside the AssetsMetaData class.
  • Also added a toString function for the AssetsMetaData class

- The totals installed for PV / Wind and Batteries were not reset before being recalculated.
- Also added a toString function for the AssetsMetaData class
- Added a collection c_parentCoops to the GC canvas. Filled with any coops that the GC is a member of. (not customers)
- Removed the updates to totalInstalled PV / Wind / Batteries from the AssetsMetaData update method. They should always already be correctly set.
- Now sets the totalInstalled PV / Wind / Batteries in the AssetsMetaData class of the EnergyCoop.
@Luc-Sol
Copy link
Contributor Author

Luc-Sol commented Mar 20, 2025

Added another commit where I removed the recalculation in the method entirely. Also added the bookkeeping for the EnergyCoop totals (requires Zenmo/zero_Interface-Loader#67 )

@AteZenmo
Copy link
Collaborator

AteZenmo commented Mar 21, 2025

gaat iets nog niet helemaal goed bij het bijhouden van c_coopParent na het verwijderen van een coop:
afbeelding

@AteZenmo
Copy link
Collaborator

Alright, fout zat hem in de lijn bij de initializatie van de customCoop: c_memberGridConnections = gcList (waar gcList de input was van de f_addCustomCoop. Waar gcList in de interface werd gevuld met de c_selectedGridConnections. Maar als op de interface nu de c_selectedGridConnections werd aangepast, paste hij de c_memberGridConnections ook automatisch aan in de coop in de engine. Dit moet niet. Dus: veranderd in c_memberGridConnections.addAll(gcList); Nu is het probleem verholpen, en zijn het weer twee apparte collecties

@AteZenmo AteZenmo merged commit 3902f28 into main Mar 21, 2025
1 of 2 checks passed
@AteZenmo AteZenmo deleted the total_PV_Installed_Bug branch March 21, 2025 10:22
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