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.

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

Add feature to select 1/3/auto phases to be used in ECO+ mode #509

Closed
jwillemsen opened this issue Apr 4, 2024 · 14 comments · Fixed by #533
Closed

Add feature to select 1/3/auto phases to be used in ECO+ mode #509

jwillemsen opened this issue Apr 4, 2024 · 14 comments · Fixed by #533

Comments

@jwillemsen
Copy link
Contributor

jwillemsen commented Apr 4, 2024

Describe the solution you'd like
The latest zappi app has support to set the number of phases to be used in ECO+ mode (see https://myenergi.info/app-update-3-3-1-t12674.html#p122029)., can be 1, 3, or auto

Additional context
See CJNE/pymyenergi#28 for the pymyenergi related issue

@heikone
Copy link

heikone commented Apr 4, 2024

Would be a great feature, because the integrated solution of Zappi, when setting is "auto", is sometimes not a good choice. In my case and at partly clouded weather, the Zappi is spending more time waiting to change from one to three phase and back, than it would charge constantly at one phase. Especially, when other consumers switch on and off.
Greetings, Heiko

@jwillemsen
Copy link
Contributor Author

I have asked myenergi on the API, no feedback yet

@G6EJD
Copy link

G6EJD commented May 3, 2024

That’s because you can’t switch phases.

@G6EJD
Copy link

G6EJD commented May 8, 2024

This can’t be achieved in a Zappi or any other EVSE, don’t should be closed

@jwillemsen
Copy link
Contributor Author

jwillemsen commented May 8, 2024

The myenergi app does provide the ability to select 1/3/auto phase charging in ECO+ mode, a recent addition of the app, see the next image, it is in dutch, faseomschakeling is phase switching, you can select automatic, single phase, or three phase

@jwillemsen
Copy link
Contributor Author

Schermafbeelding 2024-05-08 om 15 41 58

@jwillemsen jwillemsen changed the title Add feature to set the phases to be used in ECO+ mode Add feature to select 1/3/auto phases to be used in ECO+ mode May 8, 2024
@G6EJD
Copy link

G6EJD commented May 8, 2024

Yes, but titles need to be clear and now it has been changed!

@jwillemsen
Copy link
Contributor Author

I will implement this asap

@phhoef
Copy link

phhoef commented Jul 9, 2024

not sure if it helps. I monitored the requests and found out the command for phase switching.
I mentioned it in the other GitHub project describing the API:
twonk/MyEnergi-App-Api#25

@heikone
Copy link

heikone commented Jul 9, 2024

Dear all,
just want to ask about the progress of this?
Background is, that the automatic switching by Zappi does not work very well (sometimes the system is taking more time switching than loading :-).
Therefore, my plan is do something more clever with automations from Homeassistant. For example much longer retention time of phase before switching from 1 to 3-phase loading.
Anyway, this requires the possibility to switch in HA.

Does anybody observe the same problem with Zappi's automatic switching?
Any advice, if setting in Zappi can be adjusted?

Best regards,
Heiko

@jwillemsen
Copy link
Contributor Author

The code necessary is ready for review on my PR.

@erikgeurts
Copy link

The code necessary is ready for review on my PR.

For some reason, one of the automatic checks is failing (pre-commit). I am not sufficiently experienced with Github to figure out which file(s), and what's the reason it fails. Perhaps have a look at that, because it seems that once this check is OK, the PR could be pushed through.

@jwillemsen
Copy link
Contributor Author

I think two linters are making opposite changes, also happens in the scheduled action, so not related to my PR

@erikgeurts
Copy link

I think two linters are making opposite changes, also happens in the scheduled action, so not related to my PR

Since I don't even know what a 'linter' is, I can't help with that. Is there someone else who could look into this and figure out what's causing the check(s) to fail? Perhaps the person who created the scheduled actions.

@CJNE CJNE closed this as completed in #533 Aug 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants