Skip to content

Add documentation for Bed Availability Prediction query - #135

Open
sonzsara wants to merge 5 commits into
mainfrom
ENG-516
Open

Add documentation for Bed Availability Prediction query#135
sonzsara wants to merge 5 commits into
mainfrom
ENG-516

Conversation

@sonzsara

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.md documenting 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.

Comment thread Internal/Prediction/bed_availability_prediction.md
Comment thread Internal/Prediction/bed_availability_prediction.md
Comment thread Internal/Prediction/bed_availability_prediction.md

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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_threshold alone controls when a ward/day is flagged. Setting admission_multiplier and turnover_multiplier to 1 does 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.

Comment thread Internal/Prediction/bed_availability_prediction.md Outdated
…rences for improved admissions and discharges calculations

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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_threshold is inaccurate: the threshold is controlled only by bottleneck_threshold (not by the admission/turnover multipliers). Setting multipliers to 1 does not make the threshold effectively 100%; only setting bottleneck_threshold to 1 would 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.

Comment thread Internal/Prediction/bed_availability_prediction.md

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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_multiplier to 1 changes the bottleneck threshold. The threshold depends only on bottleneck_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.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 on bottleneck_threshold; setting multipliers to 1 does 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants