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

Deye Microinverter is sending the same data with update period of 60s or 5m #84

Open
Steinbichler opened this issue Aug 15, 2024 · 119 comments
Assignees
Labels
enhancement New feature or request invalid This doesn't seem right question Further information is requested

Comments

@Steinbichler
Copy link

First thank you for this fine solarman integration! Great Job!

I am running homeassistant in actual version. My converter is a Deye 600. Relais is installed.

I have your ha-slarman up an running installed via HACS. Nearly everthing works fine but:

I have requested at Deye support to set the update interval for my Deye 600 to 1 Minute.
They agreed to have it done. I see in the Homepage of the converter the update time is set to 1.

But nothing changed in the ha-solarman. I changed the number in the definition file: deye_2mppt.yaml.
After the installation the update_interval: 5 was set.
What ever the number I enter, and for sure I did a restart after the change, nothing happens.
The update interval is always 5 Minutes.

Can you help please?

Thank you an regards Roland

@Steinbichler Steinbichler added the bug Something isn't working label Aug 15, 2024
@davidrapan
Copy link
Owner

Hi @Steinbichler, I'm little bit worried that you are mixing different things here together.

I have requested at Deye support to set the update interval for my Deye 600 to 1 Minute.
They agreed to have it done. I see in the Homepage of the converter the update time is set to 1.

This integration and HA has nothing to do with Deye and their support and is completely separated thing. No configuration from Deye side won't affect this integration at all.

And at the same time nothing set in this integration will affects Deye Cloud nor Solarman App. Again it is completely separate thing from those.

But nothing changed in the ha-solarman. I changed the number in the definition file: deye_2mppt.yaml.
After the installation the update_interval: 5 was set.
What ever the number I enter, and for sure I did a restart after the change, nothing happens.
The update interval is always 5 Minutes.

What are you trying to achieve? update_interval: 5 in the 2mppt.yaml means 5 seconds not 5 minutes.

@davidrapan davidrapan added question Further information is requested invalid This doesn't seem right and removed bug Something isn't working labels Aug 15, 2024
@Steinbichler
Copy link
Author

Hi if set to 5 as ist should be 5 seconds I have this:
Bildschirmfoto vom 2024-08-15 16-26-09
Update was 6 Minutes before.

@davidrapan
Copy link
Owner

davidrapan commented Aug 15, 2024

What shows Update Interval sensor in diagnostic card in device detail view?

Edit: Please enable debug logging, let it run for a while and then post the log file here please.

@CrazyUs3r
Copy link
Contributor

@Steinbichler
Wie gut ist das wlan Signal
Mein m80 macht auch nur alle 5min ist aber auch auf 1min gestellt. In der solarman app kommen die Signale aber auch nur im 5min intervall an

@Steinbichler
Copy link
Author

@davidrapan
Bildschirmfoto vom 2024-08-15 19-01-07

do you mean this one

@CrazyUs3r
Das Wlan kann das Problem nicht sein es ist sehr gut und in der Solarman App kommt der update nach einer Minute

@Steinbichler
Copy link
Author

I started debug and I will upload the file later later

@Steinbichler
Copy link
Author

home-assistant_solarman_2024-08-15T17-36-23.029Z.log

I hope it helps.Thank you an Regards Roland

@davidrapan
Copy link
Owner

davidrapan commented Aug 15, 2024

What the actual f*ck is happening in there!

Do you have it updated to v24.08.14 version? If no please do the update.

And could I please ask you to do the following please:

  1. Disable debug logging (if you have it enabled)
  2. Restart Home Assistant
  3. Remove the device from the integration
  4. Enable debug logging
  5. Add your device back again and go through configuration
  6. Let it run for some time
  7. Disable debug logging and post here the log file again

Thank you for your cooperation! 😉

@PV-Joe
Copy link

PV-Joe commented Aug 15, 2024

I remember this behavior after my first installation.

The Update information in seconds or minutes from Home Assistant was showing wrong values but the sensors updates as expected.

I haven’t done anything about it and wondered. The problem desolved by itself after a while.

@Steinbichler
Copy link
Author

Sorry about the confusion in the deleted comment.
It was a problem with my HO installation everything woks fine I will upload the log asap.
Regards Roland

@CrazyUs3r
Copy link
Contributor

@Steinbichler
Copy link
Author

Steinbichler commented Aug 16, 2024

@davidrapan
Copy link
Owner

@Steinbichler and when you are observing the value does it really update with such a long period? Or just the HA shows it incorrectly?

@davidrapan
Copy link
Owner

Maybe fix: Proper implementation of sensor platform will do something about that strange update behavior.

@CrazyUs3r
Copy link
Contributor

I test it also

@Steinbichler
Copy link
Author

Steinbichler commented Aug 16, 2024

@davidrapan.

I did a test to cover my pannels. And yes ist last at least the 5 to 6 Minutes till I see the change in HA.

A question to my understanding: Even if the content of the register in my inverter did not change, the update in HA-solarman should read the unchanged content after the interval time? Is it not in this way?

And for the link in your comment to the definition file? Should I change it like it is in the description? and clear all the names?

@davidrapan
Copy link
Owner

davidrapan commented Aug 16, 2024

I did a test to cover my pannels. And yes ist last at least the 5 to 6 Minutes till I see the change in HA.

Fuu, that is really strange :-/

A question to my understanding: Even if the content of the register in my inverter did not change, the update in HA-solarman should read the unchanged content after the interval time? Is it not in this way?

It will be read from the inverter. BUT! Ha won't update it's "Last updated" until the value really changes.

So the value just stays the same without any visible update trigger and soon as the new values differs from displayed it updates the state. And refreshes "Last changed/updated"

And for the link in your comment to the definition file? Should I change it like it is in the description? and clear all the names?

It was link to a commit, it affects whole source code not just definition profile.

@CrazyUs3r
Copy link
Contributor

CrazyUs3r commented Aug 16, 2024

I now it. But slow wifi. So my ha view
20240816_134117

20240816_134842

@davidrapan
Copy link
Owner

davidrapan commented Aug 16, 2024

@CrazyUs3r, you did not wait long enough for the file to finish uploading, hehe.

You have to wait few seconds until the "Uploading" from the name does not disappear. That indicates that the upload finished.

@davidrapan
Copy link
Owner

@Steinbichler Try v24.08.17 and see if it won't fix your issue.

@davidrapan davidrapan added bug Something isn't working and removed invalid This doesn't seem right question Further information is requested labels Aug 17, 2024
@davidrapan davidrapan self-assigned this Aug 17, 2024
@bjoernmayer
Copy link

bjoernmayer commented Aug 19, 2024

Heyho,
I am on 24.08.17 and I can see the same behaviour. Data is updated around every 5ish minutes

@davidrapan
Copy link
Owner

Hey @bjoernmayer. Please enable debug logging and post the log here.

@davidrapan
Copy link
Owner

davidrapan commented Dec 17, 2024

You can check through debug logging that data are fetched every 5 seconds but microinverter sends the same data (for PV sensors and some other) for duration of 5m (in some cases 1m). So this is not something which can be solved from the integration (or at least I don't know yet how could be device influenced in this matter).

@Adminius
Copy link

that's weird. I can't really check if cloud's 1 min really 1 min or also 5... hm.
This makes all my logics fail on cloudy days (EV charging on PV)

@davidrapan
Copy link
Owner

I totally understand! It doesn't make sense at all but I currently do not have any microinverter unit to test things out and maybe find a solution. I do plan to acquire some as soon I'll have some spare funds to "waste" to do exactly that. 😉

@Adminius
Copy link

Can I somehow help?
Theoreticaly I can provide VPN to one of my inverters

@davidrapan
Copy link
Owner

davidrapan commented Dec 18, 2024

Not really sure if VPN would be enough as my idea is to sniff out traffic between the device and the internet in MITM style and ask support to reconfigure that update interval. It could hopefully reveal something if they change anything on the device.

@Adminius
Copy link

@davidrapan
Copy link
Owner

That is for disconnecting the device from the cloud and not losing time synchronization in the process (can be done using this integration + HA automation to set time too).

@davidrapan
Copy link
Owner

Can someone try feat: Implemented response to V5 HEARTBEAT?

It should solve time synchronization for cloudless setup.

@terzo33
Copy link

terzo33 commented Dec 18, 2024

Can someone try feat: Implemented response to V5 HEARTBEAT?

It should solve time synchronization for cloudless setup.

@davidrapan how can I do?

@davidrapan
Copy link
Owner

davidrapan commented Dec 18, 2024

Just run the latest code.

Meaning you have to manually update the code as it's not part of any release yet.

@terzo33
Copy link

terzo33 commented Dec 18, 2024

Just run the latest code.

Meaning you have to manually update the code as it's not part of any release yet.

Perfect. I did changes. Need to wait tomorrow morning, when microinverter will start to works

@davidrapan
Copy link
Owner

@terzo33, I had to do some fixes already fix: Sequence number for response to V5 HEARTBEAT 😉

@terzo33
Copy link

terzo33 commented Dec 18, 2024

Done 💪

@davidrapan
Copy link
Owner

One more fix: Run response to HEARTBEAT in asyncio.ensure_future 😉😆

@terzo33
Copy link

terzo33 commented Dec 19, 2024

Today is a rainy day... i can't try :(

@terzo33
Copy link

terzo33 commented Dec 19, 2024

One more fix: Run response to HEARTBEAT in asyncio.ensure_future 😉😆

@davidrapan it doesn't seem to work

Screenshot_20241219_101736_Samsung Internet.jpg

Edit: @davidrapan doesn't work

Screenshot_20241219_112448_Samsung Internet.jpg

@davidrapan
Copy link
Owner

davidrapan commented Dec 19, 2024

The purpose of this feature is not to anyhow help w/ update period of those entities but to enable time synchronization in cloudless environment!

I just used this thread to find someone with microinverter to test the functionality.

@terzo33
Copy link

terzo33 commented Dec 19, 2024

The purpose of this feature is not to anyhow help w/ update period of those entities but to enable time synchronization in cloudless environment!

I just used this thread to find someone with microinverter to test the functionality.

@davidrapan yes but it doesn't seem to work because the data is updated every 5 minutes as it happens via cloud. is there anything I can do to help you?

EDIT: @davidrapan maybe need to disconnect my microinverters from the cloud? (solarman app)

@davidrapan
Copy link
Owner

yes but it doesn't seem to work because the data is updated every 5 minutes as it happens via cloud. is there anything I can do to help you?

You do not understand. It has nothing to do with this.

@davidrapan
Copy link
Owner

@terzo33, show me please your debug log (hope you recorded some or still have chance to).

@terzo33
Copy link

terzo33 commented Dec 20, 2024

@terzo33, show me please your debug log (hope you recorded some or still have chance to).

@davidrapan home-assistant_solarman_2024-12-20T13-14-02.228Z.txt

@davidrapan
Copy link
Owner

davidrapan commented Dec 20, 2024

Looks like it does work:

2024-12-20 09:34:06.435 DEBUG (MainThread) [custom_components.solarman.api] [38????????] V5_HEARTBEAT: a5 01 00 10 47 02 d7 cb ea 91 e6 00 5d 15
2024-12-20 09:34:06.436 DEBUG (MainThread) [custom_components.solarman.api] [38????????] V5_HEARTBEAT RESPONSE: a5 0a 00 10 17 03 cb ea 91 e6 00 01 fe 2b 65 67 00 00 00 00 56 15

@terzo33
Copy link

terzo33 commented Dec 20, 2024

Looks like it does work:

2024-12-20 09:34:06.435 DEBUG (MainThread) [custom_components.solarman.api] [38????????] V5_HEARTBEAT: a5 01 00 10 47 02 d7 cb ea 91 e6 00 5d 15
2024-12-20 09:34:06.436 DEBUG (MainThread) [custom_components.solarman.api] [38????????] V5_HEARTBEAT RESPONSE: a5 0a 00 10 17 03 cb ea 91 e6 00 01 fe 2b 65 67 00 00 00 00 56 15

@davidrapan great, but what purpose will this thing you're doing serve? What improvement will it bring?

@davidrapan
Copy link
Owner

I told you, time synchronization for installations in a cloudless (disconnected from solarman servers) environment.

@terzo33
Copy link

terzo33 commented Dec 20, 2024

I told you, time synchronization for installations in a cloudless (disconnected from solarman servers) environment.

yes I understood this, but by disconnecting from the solarman cloud can we also have improvements on the data or extrapolate more data?

@davidrapan
Copy link
Owner

It should prevent timeouts but this functionality has nothing to do w/ subject of this thread. I already said it, that I just used this thread to announce this feature. 😉

Copy link

This issue is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale label Jan 20, 2025
@davidrapan davidrapan self-assigned this Jan 20, 2025
@davidrapan davidrapan removed the stale label Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request invalid This doesn't seem right question Further information is requested
Projects
None yet
Development

No branches or pull requests

10 participants