failure-events.rst#14
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6352b055aa
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| .. math:: | ||
|
|
||
| P_{inf,year} = 1 - (1 - p_{inf,day,normal})^{365-n_{fail}} \cdot (1 - p_{inf,day,failure})^{n_{fail}} |
There was a problem hiding this comment.
Use mixed risk for failure-day aggregation
As written, the annual aggregation applies the full-day failure probability for every failure day, so the failure_duration_minutes / 1440 fraction defined just above has no effect in this per-year formula. Following this spec would treat a 30-minute event with n_fail = 1 as a 24-hour failure day and overstate annual risk; the failure-day exponent should use the mixed daily probability (or another term that incorporates x_fail).
Useful? React with 👍 / 👎.
No description provided.