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

Soc Kia #1029

Merged
merged 6 commits into from
Aug 4, 2023
Merged

Soc Kia #1029

merged 6 commits into from
Aug 4, 2023

Conversation

Flock82
Copy link
Contributor

@Flock82 Flock82 commented Jul 11, 2023

Initialer PR

Aktuell werden Werte anscheinend wegen eines versteckten Timeouts nicht übernommen.
(Auf {modules.common.store._api:26} - {DEBUG:soc_ev0} - Raw data CarState(soc=85, range=381.0, soc_timestamp=) folgt "Save state" nur, wenn das Ergebnis innerhalb weniger Sekunden vorliegt. Beim Kia-Modul muss aber alleine 130 Sekunden auf den Server gewartet werden.)

@MartinRinas
Copy link
Contributor

MartinRinas commented Jul 12, 2023

@Flock82 hast Du herausgefunden an welcher Stelle dieser versteckte Timeout definiert wird? Ich habe das gleiche Verhalten beim Tesla SoC Modul beobachtet. Hier wird der SoC abgerufen, aber die 'save state' Meldung erfolgt nur wenn das Auto schon wach war und die Antwort schnell kommt. Muss das Auto erst aufgeweckt werden wird der SoC nicht übernommen. Daher funktioniert der regelmässige SoC Abruf im Grunde nicht, lediglich beim Anstecken & während des Ladens wird der SoC korrekt abgerufen.

@Flock82
Copy link
Contributor Author

Flock82 commented Jul 12, 2023

Nein, keine Ahnung wo das herkommt. Das Timing lässt vermuten, dass es evtl. nicht mehr möglich ist, den Status zu setzen, sobald der nächste Regelungsdurchlauf stattfindet, das ist aber reine Vermutung.

Comment on lines 401 to 403
log.error("kia.getDeviceId: Request device id failed")
log.debug("kia.getDeviceId: Request device id failed: " +
response, exc_info=True)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
log.error("kia.getDeviceId: Request device id failed")
log.debug("kia.getDeviceId: Request device id failed: " +
response, exc_info=True)
log.exception("kia.getDeviceId: Request device id failed")

Exceptions bitte immer loggen. Mit log.exception wird auch der Stack automatisch mitgeloggt. Bitte auch an den anderen Stellen anpassen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ist entsprechend angepasst

@Flock82 Flock82 requested a review from LKuemmel July 27, 2023 11:18
@benderl benderl added the enhancement New feature or request label Aug 4, 2023
@benderl benderl modified the milestones: 2.1.1, 2.2.0 Aug 4, 2023
@LKuemmel
Copy link
Contributor

LKuemmel commented Aug 4, 2023

Zu dem Timing-Problem bitte ein Issue erstellen. Gerne auf die Konversation hier verlinken. Ich habe eine Vermutung und würde dann einen Feature-Branch erstellen.

@LKuemmel LKuemmel merged commit 6fb9d3f into openWB:master Aug 4, 2023
@weichenb
Copy link

weichenb commented Aug 8, 2023

@Flock82 :
Danke für den ersten Entwicklungsschritt!
Abfrage funktioniert laut Log-File, Ergebnis wird aber nicht im Frontend angezeigt.

2023-08-08 22:44:42,868 - {urllib3.connectionpool:973} - {DEBUG:fetch soc_ev5} - Starting new HTTPS connection (1): prd.eu-ccapi.kia.com:8080 2023-08-08 22:44:42,868 - {urllib3.connectionpool:973} - {DEBUG:fetch soc_ev5} - Starting new HTTPS connection (1): prd.eu-ccapi.kia.com:8080 2023-08-08 22:44:43,079 - {urllib3.connectionpool:452} - {DEBUG:fetch soc_ev5} - https://prd.eu-ccapi.kia.com:8080 "GET /api/v1/spa/vehicles/dfdbda51-cd87-4d17-9fb5-d75cfc6da717/ccs2/carstatus/latest HTTP/1.1" 200 None 2023-08-08 22:44:43,079 - {urllib3.connectionpool:452} - {DEBUG:fetch soc_ev5} - https://prd.eu-ccapi.kia.com:8080 "GET /api/v1/spa/vehicles/dfdbda51-cd87-4d17-9fb5-d75cfc6da717/ccs2/carstatus/latest HTTP/1.1" 200 None 2023-08-08 22:44:43,091 - {modules.common.store._api:26} - {DEBUG:fetch soc_ev5} - Raw data CarState(soc=52, range=265.0, soc_timestamp=)

Apropos, ich habe den Sleeptimer auf 15 Sekunden reduziert. Die Abfrage funktioniert laut log trotzdem, angezeigt wird aber nichts!
2023-08-08 23:10:45,980 - {modules.vehicles.kia.api:782} - {INFO:fetch soc_ev5} - Kia/Hyundai: Waiting 130 seconds 2023-08-08 23:11:00,995 - {modules.vehicles.kia.api:785} - {INFO:fetch soc_ev5} - Kia/Hyundai: Receiving status 2023-08-08 23:11:01,011 - {urllib3.connectionpool:973} - {DEBUG:fetch soc_ev5} - Starting new HTTPS connection (1): prd.eu-ccapi.kia.com:8080 2023-08-08 23:11:01,011 - {urllib3.connectionpool:973} - {DEBUG:fetch soc_ev5} - Starting new HTTPS connection (1): prd.eu-ccapi.kia.com:8080 2023-08-08 23:11:01,267 - {urllib3.connectionpool:452} - {DEBUG:fetch soc_ev5} - https://prd.eu-ccapi.kia.com:8080 "GET /api/v1/spa/vehicles/dfdbda51-cd87-4d17-9fb5-d75cfc6da717/ccs2/carstatus/latest HTTP/1.1" 200 None 2023-08-08 23:11:01,267 - {urllib3.connectionpool:452} - {DEBUG:fetch soc_ev5} - https://prd.eu-ccapi.kia.com:8080 "GET /api/v1/spa/vehicles/dfdbda51-cd87-4d17-9fb5-d75cfc6da717/ccs2/carstatus/latest HTTP/1.1" 200 None 2023-08-08 23:11:01,281 - {modules.common.store._api:26} - {DEBUG:fetch soc_ev5} - Raw data CarState(soc=52, range=265.0, soc_timestamp=)

@LKuemmel
Copy link
Contributor

LKuemmel commented Aug 9, 2023

Die Nachricht ist nach der Nachricht im Issue #1075 obsolet? Am besten im Issue weiter behandeln.

@Flock82 Flock82 deleted the soc_kia branch August 14, 2023 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants