SAIC-iSmart-API / saic-python-mqtt-gateway

MIT License
66 stars 20 forks source link

Power Always 0kW after Upgrade to 0.6.0 #221

Closed killee closed 4 months ago

killee commented 4 months ago

After update to the version 0.6.0 the power is always 0.

With the release before everything works as expected.

Screenshot_2024-04-16-21-46-56-177_io.homeassistant.companion.android.jpg

I've done the update at April 15. In the evening. After that the power is 0kW.

Also the current seems to be always 0.

I have a MG5 SR.

nanomad commented 4 months ago

Current being 0 indeed matches power being 0. I think it's 0 because the validation flag we get from the API says the value is unreliable.

I'm assuming you are running HA. Please run this test:

Attach the dump to this bug report. There are no info exposed so it's safe to upload here

kimtatti commented 4 months ago

Same issue, but current shows -3.85A and charge power is 1.433kw. Started april 7 when updated to some pre release version and issue seems to have been constant after that. Even with 6.0 official version. These values are not in my normal HA dashboard, so just checked when this issue came.

nanomad commented 4 months ago

@kimtatti do you have an mg5 as well? Can you please run the test I mentioned above?

I need a capture of the raw messages to compare agains the mg4

kimtatti commented 4 months ago

@kimtatti do you have an mg5 as well? Can you please run the test I mentioned above?

I need a capture of the raw messages to compare agains the mg4

Yes! But next charge is probably over the weekend, so will take a few days. Facelift mg5 LR.

nanomad commented 4 months ago

If you know you are leaving home you can trigger it even before going out

Anyway I'll prepare a beta just for you to test. Are you able to run a standalone docker container? Or are you relying on the home assistant packaged addon?

Il Mer 17 Apr 2024, 19:53 kimtatti @.***> ha scritto:

@kimtatti https://github.com/kimtatti do you have an mg5 as well? Can you please run the test I mentioned above?

I need a capture of the raw messages to compare agains the mg4

Yes! But next charge is probably over the weekend, so will take a few days. Facelift mg5 LR.

— Reply to this email directly, view it on GitHub https://github.com/SAIC-iSmart-API/saic-python-mqtt-gateway/issues/221#issuecomment-2061887449, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIT3RQHKVX3LASPLFFYUH3Y52ZKZAVCNFSM6AAAAABGJ7R5XSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRRHA4DONBUHE . You are receiving this because you were assigned.Message ID: @.*** com>

kimtatti commented 4 months ago

If you know you are leaving home you can trigger it even before going out Anyway I'll prepare a beta just for you to test. Are you able to run a standalone docker container? Or are you relying on the home assistant packaged addon? Il Mer 17 Apr 2024, 19:53 kimtatti @.> ha scritto: @kimtatti https://github.com/kimtatti do you have an mg5 as well? Can you please run the test I mentioned above? I need a capture of the raw messages to compare agains the mg4 Yes! But next charge is probably over the weekend, so will take a few days. Facelift mg5 LR. — Reply to this email directly, view it on GitHub <#221 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIT3RQHKVX3LASPLFFYUH3Y52ZKZAVCNFSM6AAAAABGJ7R5XSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRRHA4DONBUHE . You are receiving this because you were assigned.Message ID: @. com>

Simple HAOS setup with add ons. One of my first real things that I have done with HA so not that much experience with these. But I can try to help as much as I can. I have a VPN setup so I can connect to the car anywhere if needed.

killee commented 4 months ago
  • saic/_internal/api/vehicle/charging/mgmtData/response

Hi nanomed, thank you very much for the fast response. I've attached the dump.

I use the Home Assistant packed addon. I can set up something in a separat docker enviroment but it will take a bit of time. Sorry for that.

mqtt_dump.txt

nanomad commented 4 months ago

Thanks a lot @killee. The response it's quite clear, on the MG5 there's no BMS Pack Current validation flag. I'll make that optional and use it if available

Thanks a lot for reporting this :)

nanomad commented 4 months ago

@killee @kimtatti 0.6.1-rc4 has been released in the beta HA addon, please install that, test and report back.

killee commented 4 months ago

Hi @nanomad, Works as respected again. Thank you very much for the fast fix and your great work!

Regards Kille