Prioritätensteuerung für Fahrzeuge

[LKuemmel]

No description provided.

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[Copilot]

Pull request overview

Copilot reviewed 29 out of 29 changed files in this pull request and generated no new comments.

Suppressed comments (3)

packages/control/counter_all/hierarchy.py:335

• In _add_missing_entries wird in der Warnmeldung element['id'] verwendet, obwohl hier ein fehlender Eintrag aus data_structure ergänzt wird. In diesem Codepfad ist element entweder der zuletzt iterierte Hierarchie-Eintrag (falsche ID) oder bei leerer Hierarchie sogar nicht definiert (UnboundLocalError). Stattdessen sollte die neu hinzugefügte ID (entry_num) verwendet werden.

                    pub_system_message({}, f"{component_type_to_readable_text(type_name)} mit ID {element['id']} wurde"

packages/control/counter_all/counter_all_data.py:104

• LoadmanagementPrioProtocol._remove_loadmanagement_prio_item ist typisiert als (entry: Dict) -> None, die Implementierung in LoadmanagementPrioMixin arbeitet aber mit List[Dict] und liefert ein bool zurück. Das bricht Typprüfungen und erschwert die Weiterentwicklung (z.B. bei mypy/pyright).

    def _remove_loadmanagement_prio_item(self, id: int, entry: Dict) -> None: ...

packages/control/counter_all/loadmanagement_prio.py:69

• sort_cps_by_loadmanagement_prios_nested gibt aktuell nur Ladepunkte zurück, deren EV-ID in loadmanagement_prios vorkommt. Wenn die Liste leer/unvollständig ist (z.B. beim ersten Start oder nach inkonsistenten Daten), werden verbleibende Chargepoints komplett ignoriert – die aufrufenden Algorithmen setzen dann für diese CPs keinen Strom mehr. Besser: alle nicht durch Prioritäten abgedeckten CPs am Ende als Fallback-Gruppe (in preferenced Reihenfolge) anhängen.

        sorted_cps = []
        for entry in self.data.get.loadmanagement_prios: