Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Electric current - 1 phase #25

Closed
NCVervaet opened this issue Nov 24, 2023 · 8 comments · Fixed by #26
Closed

Electric current - 1 phase #25

NCVervaet opened this issue Nov 24, 2023 · 8 comments · Fixed by #26
Assignees
@oh2th
Labels
APIv2 API v2 devices and functions bug Something isn't working

Comments

@NCVervaet
Copy link

Hello Tapio, first of all, thanks for the great go-e app for homey, I am exploring it, being the owner of go-e charger (gemini 4th gen).

I have a nissan leaf (2012), which I have charge at 1 phase, 3 phases is not possibl. When I use the 'when-card' 'the electric current changed' and connect it with a timeline card with the present current, it seems to work incorect: it returns 1/3 of the present current. So for instance, if the current set is 6 amperes, the 'tag' current returns 2A. Is there something I do wrong? Or is this really a bug? Thanks, Norbert
@oh2th
Copy link
Owner

oh2th commented Nov 25, 2023 •
edited
Loading

Thank you for your report. I will check this as it should come from the device directly as is.

Meanwhile, when you have the car next time connected and charging, with your browser fetch 'http://ip_of_goe/api/status?filter=sse,fna,car,alw,acs,frc,trx,err,wh,pha,nrg,eto,tma,amp,ama,cbl,fsp' and reply here with the output, preferably put the output in a code block in the message editor.

@oh2th oh2th self-assigned this Nov 25, 2023
@oh2th oh2th added bug Something isn't working APIv2 API v2 devices and functions labels Nov 25, 2023
@NCVervaet
Copy link
Author

NCVervaet commented Nov 25, 2023 via email •
edited
Loading

@NCVervaet
Copy link
Author

Thank you for your report. I will check this as it should come from the device directly as is.

Meanwhile, when you have the car next time connected and charging, with your browser fetch 'http://ip_of_goe/api/status?filter=sse,fna,car,alw,acs,frc,trx,err,wh,pha,nrg,eto,tma,amp,ama,cbl,fsp' and reply here with the output, preferably put the output in a code block in the message editor.

Hello Tapio, present I am charging at 16A, Single phase. I get returned:

{"alw":true,"sse":"220618","eto":1213072,"ama":16,"amp":16,"fna":"EV Charger ","fsp":true,"acs":0,"frc":0,"car":2,"err":0,"cbl":32,"pha":[true,false,false,true,true,true],"wh":1221.832,"trx":null,"tma":[20.5,17.5],"nrg":[230.95,237.46,237.77,4.03,16.112,0,0,3629.91,0,0,0,3629.91,97.55027,0,0,100]}

@oh2th
Copy link
Owner

oh2th commented Nov 25, 2023 •
edited
Loading

The pha value is the one that shows us that the source is single phase, however output mode is three phases. Are you using the schuko adapter?
So far there hasn't been any user with such connection.

But indeed, there is some level of a bug, as fsp is true, meaning that single phase mode has been selected. Need to check the logic.

@NCVervaet
Copy link
Author

NCVervaet commented Nov 26, 2023 via email

@oh2th
Copy link
Owner

oh2th commented Dec 3, 2023

There is a fix on how active phases are counted for the current calculation.
Could you test the test version from: https://homey.app/a/com.go-e.charger/test/

@NCVervaet
Copy link
Author

NCVervaet commented Dec 3, 2023 via email

@oh2th oh2th linked a pull request Dec 4, 2023 that will close this issue
Fix for 25 electric current 1 phase #26
Merged
@oh2th oh2th closed this as completed in #26 Dec 4, 2023
@NCVervaet
Copy link
Author

NCVervaet commented Dec 5, 2023 via email

@oh2th oh2th mentioned this issue Dec 6, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oh2th oh2th

Labels
APIv2 API v2 devices and functions bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix for 25 electric current 1 phase oh2th/com.go-e.charger
2 participants
@oh2th @NCVervaet