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

add calculate_soc to settings, needed for PSA #413

Merged
merged 1 commit into from
Feb 2, 2024

Conversation

MartinRinas
Copy link
Contributor

No description provided.

@LKuemmel
Copy link
Contributor

Ich sehe keine Notwendigkeit für diese Einstellung. Der SoC für PSA wird immer berechnet. Dies abschaltbar zu machen, führt dazu, dass SoC-geführtes laden nicht mehr funktioniert.

@MartinRinas
Copy link
Contributor Author

MartinRinas commented Jan 31, 2024

Das Thema ist ja dass die PSA Anbindung nicht mehr funktioniert, und auch auf absehbare Zeit (m.E.) nicht funktionieren wird. Was in Änderungen der PSA API begründet ist - wir können uns nicht mehr als mobile App ausgeben, die offizielle API hat andere Herausforderungen. Langes Thema... Daher die Idee alles PSA Anwender in Richtung Tronity als SoC Provider zu migrieren. Dann brauchts aber eine Möglichkeit den Tronity SoC während des Ladens zu berechnen.

@LKuemmel
Copy link
Contributor

ok, dann muss die Einstellung aber ins Tronity-Modul und nicht zu PSA.

@MartinRinas
Copy link
Contributor Author

uhm jup, da ist sie doch auch, oder hab' ich mich da vertan?

@LKuemmel
Copy link
Contributor

Oh, ich hatte im Core-Repo nach der Einstellung "calculate_soc" gesucht und da gibt es nur beim PSA-Modul eine Einstellung, die so heißt aber nichts macht.
Da hatte ich nicht auf den Ordner-Namen im PR geschaut. Sorry für die Verwirrung.

@MartinRinas
Copy link
Contributor Author

gehört zu openWB/core#1398

@LKuemmel LKuemmel merged commit b54a77b into openWB:main Feb 2, 2024
@MartinRinas MartinRinas deleted the tronity_manual_calc branch June 17, 2024 12:39
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