github-mirrors/pyPLC
1
0
Fork 0
mirror of https://github.com/uhi22/pyPLC.git synced 2024-09-11 00:38:29 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

docu: bug_analysis updated

Browse source
This commit is contained in:
uhi22 2023-03-24 07:41:30 +01:00
parent 70ef4994ca
commit ec955273fe
1 changed files with 19 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
doc/bug_analysis.md
View file

@ -1,6 +1,18 @@
# Open issues # 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". - 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". - But we ignore this, and send in all next messages (starting with ServiceDiscoveryReq) the sessionID "deadbeefdeadbeef".
- The SuperCharger correctly complains with "ResponseCode": "FAILED_UnknownSession". - 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 - implementation done with https://github.com/uhi22/OpenV2Gx/commit/9c08c19ce14446a316ed4059d6d5b5e07721fd9d
- to be tested. - to be tested.
## Issue17: Welding detection fails
## Issue 17: Welding detection fails
- WeldingDetectionRes says "failed" on alpitronics, and no response on ABBHPC and ABBTriple - WeldingDetectionRes says "failed" on alpitronics, and no response on ABBHPC and ABBTriple
- version v0.4-7-g7cea8b5 (2022-12-21) - 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 # 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 ## [Solved] Issue16: First CurrentDemandReq is rejected with FAILED_SequenceError and EVSE_Shutdown
- observed on Alpitronics HPC with version v0.4-6-g257e5af - observed on Alpitronics HPC with version v0.4-6-g257e5af
- log 2022-12-20_alpi_currentDemandSequenceErrorShutdown.pcapng - log 2022-12-20_alpi_currentDemandSequenceErrorShutdown.pcapng