search

openvehicles / Open-Vehicle-Monitoring-System-3

Open Vehicle Monitoring System - Version 3
http:///www.openvehicles.com/
Other
587 stars 225 forks source link

NL: Metrics Not Updated on wakeup #544

Open mjkapkan opened 3 years ago

mjkapkan commented 3 years ago

I noticed that a lot of metrics on NL MY 2013 don't regularly update and can remain unchanged for hours. While sending wakeup command does not make a difference, if I send start charge command (when vehicle is unplugged) it seems to update most of those metrics.

I would suggest to add the CAN frame that is sent by start charge to actively poll metrics, or otherwise at least add it to the wakeup command.

Any thoughts?

mjkapkan commented 3 years ago

can can1 tx standard 5c0 seems to do the trick.

dalathegreat commented 2 years ago

Fix for this is pending in PR #662

dalathegreat commented 2 years ago

It turned out this is not yet fixed completely. In my PR, I changed so that 0x679 is sent instead of 0x68c. This only worked when vehicle was plugged in and idle. So I'll need to do more testing with which commands are suitable for when not plugged in.

Sending 0x5c0 works, it is a bit strange, maybe the VCM listens for this message since it contains battery heating request? That might be why the vehicle wakes up when receiving it?

Bonus, the 0x679 command when plugged in and idle turns on the DC-DC converter and tops up the 12V battery, nice addition.

Thoughts @mjkapkan ?

mjkapkan commented 2 years ago

I think we can just add 0x5c0 to the wakeup command. Does not seem to do any harm ;)

dalathegreat commented 2 years ago

Fix for this is pending in PR https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/pull/671

mjkapkan commented 2 years ago

This is not yet fixed. At least for the MY 2013 when unplugged. Wakeup does not update v.e.cabintemp or v.e.temp. One trick that worked is to send 'charge start' command. So we might need to pick a couple of commands from that sequence.