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

Can’t set own ping interval #105304

Closed
danmrossi opened this issue Dec 8, 2023 · 5 comments
Closed

Can’t set own ping interval #105304

danmrossi opened this issue Dec 8, 2023 · 5 comments
Assignees

Comments

@danmrossi
Copy link

The problem

In 2023.12.0 we have lost the ability to set our own ping intervals which breaks functionality for those of use who utilise Home Assistant for operations other than making a house “smarter”

Please do not read anything I have said prior or after in a negative light as not trying to upset or step on anyone’s toes. Just trying to get my point across.

I am all for change and the move of various bits and bobs out of YAML yet there is a lot of us out there that get peeved when not all of the functionality that we had with YAML doesn’t come across and coders make their own minds up on what functionality either comes across or what settings will be changed then can’t be edited etc etc when their can be hundreds of use cases that one is not aware of.

If we can’t have individual ping intervals per sensor/entity then even a global option that we can set our own value would be a great compromise.

I do really genuinely appreciate everyone’s help to me over the years and appreciate for a lot of people coding on here is more of a love job than a paid job I just hope you read this in a positive light and hope at some point this can be rectified.

What version of Home Assistant Core has the issue?

2023.12.0

What was the last working version of Home Assistant Core?

2023.11.3

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Ping

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@danmrossi
Copy link
Author

If I have to roll back to 2023.11.3 that’s fine by me it’s just I only really updated because I wanted the fix in the front end for half created automations that you don’t save that still hang around.

@joostlek
Copy link
Member

joostlek commented Dec 8, 2023

Look, setting a custom update interval isn't an issue specific for a single integration. To avoid having every integration implement their own way of doing scan interval, we have made something that is way more flexible that will suite probably everyones need. The update_entity service.

This service gives you way more freedom than just an input field when configuring an integration.

  1. You can change the update interval whenever you want. If you have an API limit on your solar panels for example, you can now choose to only ping per hour during night and during day you poll more, giving you more data points. Or maybe you don't want the weather to update as much when you're not at home. You do you, make it fit your need.
  2. You can apply this to every integration. If every integration would have to implement a configuration to add scan interval, you would not be able to set scan interval in 95% of all integrations at this point.

So to end this topic now. You can still do your use case, except you need to do a bit of effort to make it work. But this solution is the most consistent way over all integrations, giving you more freedom and possibilities than with just an input field.

@joostlek joostlek closed this as not planned Won't fix, can't repro, duplicate, stale Dec 8, 2023
@home-assistant
Copy link

home-assistant bot commented Dec 8, 2023

Hey there @jpbede, mind taking a look at this issue as it has been labeled with an integration (ping) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of ping can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign ping Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


ping documentation
ping source
(message by IssueLinks)

@danmrossi
Copy link
Author

danmrossi commented Dec 8, 2023 via email

@joostlek
Copy link
Member

joostlek commented Dec 8, 2023

But since you have the freedom, there is no default way to do what you want. When you just want to higher or lower the interval, checkout the time interval trigger:
afbeelding

@github-actions github-actions bot locked and limited conversation to collaborators Jan 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants