[sensors] Comparison between seconds and milliseconds in "Report latest reading updated"? (#441)

rakuco has just created a new issue for https://github.com/w3c/sensors:

== Comparison between seconds and milliseconds in "Report latest reading updated"? ==
https://w3c.github.io/sensors/#report-latest-reading-updated has:

> * Let reportingInterval be the result of 1 / reportingFrequency.
> * Let timestampDelta be the result of [latest reading](https://w3c.github.io/sensors/#latest-reading)["timestamp"] - lastReportedTimestamp.
> * If timestampDelta is greater than or equal to reportingInterval
>   * [...]

`reportingFrequency` is measured in Hz, so `reportingInterval` is measured in seconds. `latest reading["timestamp"]` and `lastReportedTimestamp`, on the other hand, are high resolution timestamps measured in milliseconds.

A `timestampDelta` of 2 means a different of two milliseconds, while a sampling frequency of 1Hz means `reportingInterval` is 1 and the check above will be true even though it should not.

Please view or discuss this issue at https://github.com/w3c/sensors/issues/441 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Thursday, 6 October 2022 10:31:53 UTC