search

inform-us / requirements_specifications

A place to store all of the metrics rules in .md format
Apache License 2.0
0 stars 1 forks source link

Confirm compiance time. #7

Open thompson318 opened 2 months ago

thompson318 commented 2 months ago

Our code currently checks compliance at 13:30 not 13:00 as written here. Should we update code or this document.

https://github.com/inform-us/requirements_specifications/blob/9fcf4e1605a1c82d709aeadb8be0e2e43e44a645/rules/targets.md?plain=1#L13

nmaccallum commented 2 months ago

MD file now updated. Compliance is measured at 13:00, but we choose to run calculation at 13:30 to give the staff some leeway.

thompson318 commented 2 months ago

Thanks. I need to check this. To be precise, I think the calculations are run at 3:30 and 4:30 am each day. The database is queried for occurrences where all targets were set before 13:30 on a given day. Presumably anytime between midnight and 13:30. So it's not when the calculation is run that gives it some leeway, but rather what is run. I think the result is the same, but I just want to make sure the documentation is right.