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

[Bug]: lost sensors for buffer:( #121

Closed
d2ar3k opened this issue Dec 13, 2024 · 15 comments
Closed

[Bug]: lost sensors for buffer:( #121

d2ar3k opened this issue Dec 13, 2024 · 15 comments
Assignees
Labels
bug Something isn't working Sensor This doesn't seem right

Comments

@d2ar3k
Copy link

d2ar3k commented Dec 13, 2024

The problem

There are no sensors.for buffer like it was in v 1.03

What version of ecoNET300 integration has the issue?

1.06

What version of Home Assistant Core has the issue?

Core

What was the last working version of ecoNET300 integration?

1.03

Diagnostics information

No response

Anything in the logs that might be useful for us?

Nope

Additional information

Nope

@d2ar3k d2ar3k added the bug Something isn't working label Dec 13, 2024
@jontofront
Copy link
Owner

Could you please clarify the name of the sensor you are referring to?

@jontofront jontofront added the Sensor This doesn't seem right label Dec 14, 2024
@d2ar3k
Copy link
Author

d2ar3k commented Dec 14, 2024

sensor.plum_econet300_temperatura_bufora_dolna

And also

sensor.plum_econet300_temperatura_bufora_gorna

@jontofront
Copy link
Owner

can you send me ([email protected]) json file from these endpoinds? I will add
http://econet_local_ip/econet/regParams
http://econet_local_ip/econet/regParamsData
http://econet_local_ip/econet/sysParams
http://econet_local_ip/econet/rmCurrentDataParams
http://econet_local_ip/econet/rmCurrentDataParamsEdits

@d2ar3k
Copy link
Author

d2ar3k commented Dec 14, 2024

Just sent teough email
Chyba możemy po polsku pisac coś mi się tak wydaje

@jontofront
Copy link
Owner

Add sensors:
tempUpperBuffer
tempLowerBuffer

if null disable it

@jontofront jontofront linked a pull request Dec 14, 2024 that will close this issue
@jontofront
Copy link
Owner

sensor.plum_econet300_temperatura_bufora_dolna

And also

sensor.plum_econet300_temperatura_bufora_gorna

You can test it this version:
image
dev

@d2ar3k
Copy link
Author

d2ar3k commented Dec 15, 2024

On my way

@d2ar3k
Copy link
Author

d2ar3k commented Dec 15, 2024

Great work! Thank you. I will use this version till next official release:)

@d2ar3k
Copy link
Author

d2ar3k commented Dec 15, 2024

One other sensor is missing... HUW temperature....
You can see SET temperature but not actual

@d2ar3k
Copy link
Author

d2ar3k commented Dec 15, 2024

Its "tempCWU"

@d2ar3k
Copy link
Author

d2ar3k commented Dec 15, 2024

So need to go back to beta version because of this sensor missing:(

But big thanks for your support. Great work

@d2ar3k
Copy link
Author

d2ar3k commented Dec 15, 2024

Also noticed that sensor updates on beta version 1.03 are wayyy better than on dev version...

@jontofront
Copy link
Owner

tempCWU

ok I add Water heater temperature - tempCWU
Tommorow realise it

@jontofront
Copy link
Owner

One other sensor is missing... HUW temperature....
You can see SET temperature but not actual

maybe you can make print screen?

@jontofront
Copy link
Owner

Also noticed that sensor updates on beta version 1.03 are wayyy better than on dev version...

That might be the case, but I had to rewrite everything and clean up the old unused code. This was necessary because there are many devices with different versions. Now, it will be more flexible to continue working. Apologies for any inconvenience caused.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Sensor This doesn't seem right
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants