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

More conservative defaults for mqtt-hassio.cfg as some messages might not like being polled #1213

Closed
wants to merge 2 commits into from

Conversation

pulquero
Copy link

@pulquero pulquero commented Mar 3, 2024

Fixes #1205. Not seeing any issues with this and the current csvs. Maybe in the future there needs to be filter-seen-include/exclude to fine-tune automatic polling.

@pulquero
Copy link
Author

pulquero commented Mar 4, 2024

I'm going to come up with something better than this, as this is probably be quite a disruptive change - turning off polling. I'm going to try a suitable default set of messages which can be safely polled with automatic polling and set filter-name to that.

@pulquero pulquero changed the title Use filter-seen = 1 for mqtt-hassio.cfg as some messages might not like being polled More conservative defaults for mqtt-hassio.cfg as some messages might not like being polled Mar 4, 2024
@pulquero
Copy link
Author

pulquero commented Mar 4, 2024

Using a consistent naming convention, i.e. that used in john30/ebusd-configuration#407. This default combination of messages to poll doesn't cause any strange behaviour.

@john30
Copy link
Owner

john30 commented Mar 24, 2024

indeed this would be rather disruptive and as such it is declined.

the original problem #1205 is completely different and can be easily solved on the message definitions by using a different user level for isolating these monitoring messages that probably only rare users like to have automatically activated for poll by HA (despite the fact that these destabilize the system as it seems)

@john30 john30 closed this Mar 24, 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 this pull request may close these issues.

Has 23.3 stopped my heat pump running properly? - 5kW Arotherm Plus
2 participants