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

DDF for Eva Powermeter #7034

Merged
merged 2 commits into from Jul 20, 2023
Merged

Conversation

sevrinrobstad
Copy link
Contributor

Fix #6343 .

@BabaIsYou
Copy link
Contributor

Hi,
To avoid some isuues it's preferable that refresh.interval would be higher than max in binds, as a last resort if bind failed.
And these values (same for default": 0) are already into generic items for "power", "current" and "voltage" then useless into the DDF.

@sevrinrobstad
Copy link
Contributor Author

Hi, To avoid some isuues it's preferable that refresh.interval would be higher than max in binds, as a last resort if bind failed. And these values (same for default": 0) are already into generic items for "power", "current" and "voltage" then useless into the DDF.

OK so just remove the refresh.interval and default:0 all together?

@BabaIsYou
Copy link
Contributor

Exactly, or just default and put a higher value for refresh like 360 for instance ;-)

@sevrinrobstad
Copy link
Contributor Author

Updated.

@manup manup added this to the v2.23.0-beta milestone Jul 20, 2023
@manup manup changed the title DDF - for Eva Powermeter DDF for Eva Powermeter Jul 20, 2023
@manup manup merged commit 0747d53 into dresden-elektronik:master Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EVA Han Powermeter
3 participants