Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI] Run nightly calibration with new healthsystem mode #1357

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

giordano
Copy link
Member

@giordano giordano commented May 17, 2024

This turn the old calibration workflow into a reusable workflow, which can be used by multiple healthsystem modes runs.

This will eventually fix #1335.

As discussed in the meeting this morning we will probably go with a single larger machine (Standard_F32s_v2?), to answer the question at #1335 (comment).

TODO:

  • reference mode name in WORKTREE_PATH and ENV, so that the different calibration runs don't step on each other (especially the cleanup stage)
  • the run script should actually do something different for the two different modes (right now I don't know exactly where this should happen)
  • set up and provision the new virtual machine. Remember to change the labels of the runners, some (10?) runners will be only for mode1, other only for mode2 (but both for tasks), and one for postprocess
  • change the runs-on to use the new machine name
  • the postprocessing script will have to be adapted
  • the html generating script will have to be adapted
  • ...

@giordano giordano changed the title [CI] Run nightly calibration with new model [CI] Run nightly calibration with new healthsystem mode Jun 4, 2024
@giordano giordano force-pushed the mg/calibration-mod2 branch 6 times, most recently from 35fb6d5 to 71006e7 Compare June 4, 2024 15:50
This turn the old calibration workflow into a reusable workflow, which can be
used by multiple model runs.
@giordano giordano force-pushed the mg/calibration-mod2 branch 7 times, most recently from 2215619 to 0137d4d Compare June 4, 2024 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Do scaling runs for mode_appt_constraints = 2 too
1 participant