Re: [compute-pressure] Fix penalty algorithm (#230)

There is still an issue with the algorithm.

Assuming we are reaching the amount of changes allowed during the observation window and we do not pass the rate_obfuscation_test in  https://www.w3.org/TR/compute-pressure/#data-delivery
we are reaching 6.2.2 AFTER penalty, which will imply that "Set observer.[[LastRecordMap]][source] to record." will be called after the penalty time is over.

This means that we will get new updates, which will pass has change in data test.
https://www.w3.org/TR/compute-pressure/#dfn-has-change-in-data, because LastRecordMap wasnt updated.
This will imply that the count of change data will increase by 1, and pass all the other tests, though we are in penalty.
So it delays the penalty by 1 state change.



-- 
GitHub Notification of comment by arskama
Please view or discuss this issue at https://github.com/w3c/compute-pressure/pull/230#issuecomment-1695269599 using your GitHub account


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

Received on Monday, 28 August 2023 08:32:36 UTC