From ec955273fe13648877bf681b9a64ba96286cde34 Mon Sep 17 00:00:00 2001 From: uhi22 Date: Fri, 24 Mar 2023 07:41:30 +0100 Subject: [PATCH] docu: bug_analysis updated --- doc/bug_analysis.md | 22 +++++++++++++++++++--- 1 file changed, 19 insertions(+), 3 deletions(-) diff --git a/doc/bug_analysis.md b/doc/bug_analysis.md index a7e1e9b..fde13f6 100644 --- a/doc/bug_analysis.md +++ b/doc/bug_analysis.md @@ -1,6 +1,18 @@ # Open issues -## Issue 18: On SuperCharger we ignore the session ID +## Issue21: Missing fields in ChargeParameterDiscoveryReq +- In the ChargeParameterDiscoveryReq, some fields are not filled. This leads to abort on certain charger types. +- Discussion in https://openinverter.org/forum/viewtopic.php?p=54696#p54696 +- Improvement in https://github.com/uhi22/OpenV2Gx/commit/4b5df391d56e15b45652605fa7ad8a7712e2acaf + +## Issue19: Wrong determination of MAC and IPv6 +- On linux, the deprecated ifconfig is not a good choice. And, if multiple interfaces are present, the wrong address is fetched. +- Solution: Three parts +1. Use "ip addr" instead of "ifconfig" -> done +2. Filter for the correct interface -> done +3. Make the interface name configurable -> todo + +## Issue18: On SuperCharger we ignore the session ID - In the SessionSetupResponse, the SuperCharger V3 correctly provides a sessionID, e.g. "06ef0071". - But we ignore this, and send in all next messages (starting with ServiceDiscoveryReq) the sessionID "deadbeefdeadbeef". - The SuperCharger correctly complains with "ResponseCode": "FAILED_UnknownSession". @@ -14,8 +26,7 @@ use a dynamic length instead of the fixed length LEN_OF_SESSION_ID (=8). - implementation done with https://github.com/uhi22/OpenV2Gx/commit/9c08c19ce14446a316ed4059d6d5b5e07721fd9d - to be tested. - -## Issue 17: Welding detection fails +## Issue17: Welding detection fails - WeldingDetectionRes says "failed" on alpitronics, and no response on ABBHPC and ABBTriple - version v0.4-7-g7cea8b5 (2022-12-21) @@ -28,6 +39,11 @@ and after one second the state machine seems to run into timeout. # Closed issues +## [Solved] Issue20: Missing EVCCID in the SessionSetupReq +- In the SessionSetupReq, the charger expects the EVCCID filled with the cars MAC. This is missing, and leads to abort on certain charger types. +- Discussion in https://openinverter.org/forum/viewtopic.php?p=54667#p54667 +- Fixed with https://github.com/uhi22/OpenV2Gx/commit/fc46c3ca802f08c57120a308f69fb4d1ce14f6b6 and https://github.com/uhi22/pyPLC/commit/9b39bff85a04071e5d92d613197422033f0c1d8d + ## [Solved] Issue16: First CurrentDemandReq is rejected with FAILED_SequenceError and EVSE_Shutdown - observed on Alpitronics HPC with version v0.4-6-g257e5af - log 2022-12-20_alpi_currentDemandSequenceErrorShutdown.pcapng