dalathegreat / Nissan-env200-Battery-Upgrade

Software for upgrading a 24kWh env200 to 40kWh with a Muxsan CAN-bridge
GNU General Public License v3.0
24 stars 4 forks source link

Charge timer and charge limit no longer working #4

Open phoenixs3 opened 1 year ago

phoenixs3 commented 1 year ago

After upgrading from 24kwh to 40kwh, the 80% charge limit and the charge timer no longer seem to function. Have these been successfully tested by someone else with this code running?

glynhudson commented 1 year ago

Yes, I can confirm the dashboard 80% charge limit doesn't work on my upgraded 40kWh e-NV200 either. I've not tested the timer I've never used it. I presume the 80% charge limit was a feature of the 24kWh battery, since stock 40kWh e-NV200 do not have an 80% charge limit either.

However, not having a charge limit is not a big issue since OVMS now supports remotely stopping a charge and setting a charge limit.

dalathegreat commented 1 year ago

I can confirm, that the newer battery packs don't understand the 80% longlife option at all. However, it would be possible to code this into the CAN-bridge for users that don't have OVMS.

The 0x1F2 message sent by the VCM contains a parameter called "TargetChargeSOC". This bit has two values, 0=100% and 1=80%. In theory, we could quite simply intercept this, and force a charge stop via the same mechanic as OVMS does.

Seb / Glyn, incase anyone of you wants to test this, let me know and I can code up a test.

phoenixs3 commented 1 year ago

I would be happy to test this since my Mums van doesnt have OVMS and she wouldnt want to add it. Only catch is the van is up in Scotland, so I wouldnt be able to apply the SW update until I go and see her next (likely around 1 months time) However if your able to add the software feature in I can flash it as soon as i am up with the van. Also any idea why the charge timer doesnt work?

glynhudson commented 1 year ago

Seb / Glyn, incase anyone of you wants to test this, let me know and I can code up a test.

Sure, I'm happy to help test. Getting rid of the 80% limit was a silly move by Nissan IMO

dalathegreat commented 1 year ago

I created a branch for this feature (https://github.com/dalathegreat/Nissan-env200-Battery-Upgrade/commit/3417a4dbfa320f73d8f1021423add45b687f41c6)

phoenixs3 commented 1 year ago

Thanks I will provide feedback when I can test it!

dalathegreat commented 3 months ago

We 100% fixed this on the LEAF upgrades, here is the same fix applied to env200! https://github.com/dalathegreat/Nissan-env200-Battery-Upgrade/pull/7