Conversation
There was a problem hiding this comment.
Pull request overview
Adds a new markdown documentation page for an internal “Bed Availability Prediction” query, describing purpose, parameters, and the SQL used to forecast ward-level bed occupancy over the next 7 days.
Changes:
- Added
Internal/Prediction/bed_availability_prediction.mddocumenting the forecast query, including purpose, parameters, notes, and SQL. - Introduced Metabase parameters (
admission_multiplier,turnover_multiplier,bottleneck_threshold) and explanatory notes for tuning.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…discharges calculation
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
Internal/Prediction/bed_availability_prediction.md:18
- The note is inaccurate:
bottleneck_thresholdalone controls when a ward/day is flagged. Settingadmission_multiplierandturnover_multiplierto1does not make the threshold behave like 100%.
> **Note:** `bottleneck_threshold` is a **fraction**, so enter `0.90`, not `90`. If all three variables are set to `1`, the threshold is effectively `100%`, meaning nothing will flag until a ward is completely full — set it to `0.9` (or your desired fraction) instead.
…rences for improved admissions and discharges calculations
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
Internal/Prediction/bed_availability_prediction.md:18
- The note about
bottleneck_thresholdis inaccurate: the threshold is controlled only bybottleneck_threshold(not by the admission/turnover multipliers). Setting multipliers to1does not make the threshold effectively 100%; only settingbottleneck_thresholdto1would do that.
> **Note:** `bottleneck_threshold` is a **fraction**, so enter `0.90`, not `90`. If all three variables are set to `1`, the threshold is effectively `100%`, meaning nothing will flag until a ward is completely full — set it to `0.9` (or your desired fraction) instead.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (1)
Internal/Prediction/bed_availability_prediction.md:18
- The note incorrectly implies that setting
admission_multiplier/turnover_multiplierto 1 changes the bottleneck threshold. The threshold depends only onbottleneck_threshold; multipliers only affect the forecasted occupancy numerator.
> **Note:** `bottleneck_threshold` is a **fraction**, so enter `0.90`, not `90`. If all three variables are set to `1`, the threshold is effectively `100%`, meaning nothing will flag until a ward is completely full — set it to `0.9` (or your desired fraction) instead.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (1)
Internal/Prediction/bed_availability_prediction.md:18
- The note incorrectly implies the bottleneck behavior depends on all three variables being set to
1. The flagging threshold depends only onbottleneck_threshold; setting multipliers to1does not make the threshold "effectively 100%".
> **Note:** `bottleneck_threshold` is a **fraction**, so enter `0.90`, not `90`. If all three variables are set to `1`, the threshold is effectively `100%`, meaning nothing will flag until a ward is completely full — set it to `0.9` (or your desired fraction) instead.
No description provided.