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

[Documentation] Definition of SUBSCRIPTION_MAX_INTERVAL_PUBLISHER_LIMIT vs ICD #31374

Closed
Apollon77 opened this issue Jan 11, 2024 · 1 comment
Labels
documentation Improvements or additions to documentation needs triage

Comments

@Apollon77
Copy link
Contributor

Apollon77 commented Jan 11, 2024

Documentation issues

The specificiation tells:

SUBSCRIPTION_MAX_INTERVAL_PUBLISHER_LIMIT defines the upper limit for the publisher-selected maximum interval for any subscription.
◦ If the publisher is an ICD, this SHALL be set to the Idle Mode Interval.
◦ If not, this SHALL be set to 60 minutes.

In my understand this means:

  • an ICD device (aka mostly Battery powered ones with limited power) define an "Idle Mode Interval". An Eve Door&Window as example - ok this one does not have ICD right now - but uses SII=10s which I consider comparable. This means that formally this device "SHALL" have a maximum subscription interval of 10s ...
  • any non ICD device (mostly also non-battery powered with many power resources) should set max to 60minutes nd may decide for shorter intervals

So when a controller requests 120s from Eve Door&Window then the device formally need to respond with 10s as max interval (unless the controller wants a higher max timeframe) which is the worst case for battery inn my eyes. SO in this case it is completely controller bound how long the battery holds.
Should not - especially for battery devices - the device have some more options to "protect" the battery life best possibly?

Is this really wanted that way?

Or is the Idle Interval in ICD not "the same" as SII?

Platform

No response

Anything else?

No response

@Apollon77 Apollon77 added documentation Improvements or additions to documentation needs triage labels Jan 11, 2024
@Apollon77
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation needs triage
Projects
None yet
Development

No branches or pull requests

1 participant