Skip to content

Added SELL options to TOU❘Deye 3P#1044

Open
StRobiEst wants to merge 1 commit into
davidrapan:mainfrom
StRobiEst:patch-1
Open

Added SELL options to TOU❘Deye 3P#1044
StRobiEst wants to merge 1 commit into
davidrapan:mainfrom
StRobiEst:patch-1

Conversation

@StRobiEst
Copy link
Copy Markdown

Added "Sell" options to TOU Programs

Added "Sell" options to TOU Programs

Signed-off-by: StRobiEst <22235571+StRobiEst@users.noreply.github.com>
@StRobiEst StRobiEst changed the title SELL options to TOU deye_p3.yaml Added SELL options to TOU❘Deye 3P Apr 13, 2026
@Saboter78
Copy link
Copy Markdown

Deye 1p is need this modes too

@Sopsy
Copy link
Copy Markdown
Contributor

Sopsy commented Apr 20, 2026

The modbus manuals are not the easiest to understand, but as I am reading them, this would only apply to the LV inverters and would be undefined behavior in the HV firmware. HV should use the register 146. I'd recommend confirming and limiting to LV only if so and if 146 does not work on LV.

@StRobiEst
Copy link
Copy Markdown
Author

Tested on SUN-15K-SG01HP3-EU-AM2 (High Voltage) with firmware version 3001-F052-1E08 (HMI: 2001-C030)
My Deye screen does not have the sell option, but the DeyeCloud app does. And with this commit it now works with HomeAssistant. I don't know what it will do with Grid+Sell option, but the selling works.
TOU
TOU P3

@Sopsy
Copy link
Copy Markdown
Contributor

Sopsy commented Apr 21, 2026

You don't have this "Sell" checkbox? Selling works regardless of that though, by using the "Export first" Work Mode.
Untitled

However, this is a good catch. It seems that the "Sell" checkbox does indeed toggle bit 5 of that register and I can verify your change works on the HV (FW 3002-1095-11410 / 2001-C048).

Though I personally would prefer separating this into its own toggle ("Program x Sell") as the list is getting complex. I'm not sure if selects support bit masks on writes, but if not, it would mean separating grid and gen into their own switches as well. Which would be better anyway if you ask me, and in-line with the inverter UI.

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 this pull request may close these issues.

3 participants