Skip to content

Comments

updated mod allocation in load scenario so hhd intervention aligned w…#176

Merged
morganle-48 merged 3 commits intomainfrom
change-hhd-logic
Feb 12, 2026
Merged

updated mod allocation in load scenario so hhd intervention aligned w…#176
morganle-48 merged 3 commits intomainfrom
change-hhd-logic

Conversation

@morganle-48
Copy link
Collaborator

…ith clinical ask.

Closes #175

@morganle-48 morganle-48 requested a review from yiwen-h as a code owner February 11, 2026 09:21
@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.77%. Comparing base (a964c47) to head (081bf00).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #176   +/-   ##
=======================================
  Coverage   84.77%   84.77%           
=======================================
  Files           9        9           
  Lines         703      703           
  Branches       78       78           
=======================================
  Hits          596      596           
  Misses         98       98           
  Partials        9        9           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@yiwen-h yiwen-h left a comment

Choose a reason for hiding this comment

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

Looks good to me! I haven't checked the underlying values of course - have you validated with real data, does it improve the model performance?

This made me open #177 as well

@morganle-48 morganle-48 merged commit 7a81daa into main Feb 12, 2026
6 checks passed
@morganle-48 morganle-48 deleted the change-hhd-logic branch February 12, 2026 15:18
@morganle-48
Copy link
Collaborator Author

This one is less to do with data and more to do with intervention, which we can't fully validate - but the behaviour looks sensible and as expected from what I can see. Will have a look at the new issue thanks!

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.

Change to HHD logic

2 participants