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

MOBS_NEU-GLEAM_COVID Submission #276

Merged
merged 1 commit into from
May 8, 2024

Conversation

jessica-davis
Copy link
Contributor

Description

  • Team name: MOBS_NEU
  • Model name that is being updated: GLEAM_COVID

Copy link

github-actions bot commented May 7, 2024

Run validation on files: 2024-04-28-MOBS_NEU-GLEAM_COVID.gz.parquet
End of validation check: all the validation checks were successful

@LucieContamin
Copy link
Collaborator

Hi @jessica-davis ,

Thank you for the submission. I just wanted to check I understand correctly the sample ID numbering: each model runs has different run_grouping (model run independent) for every stochastic_run group by age_group, horizon, and sub-group of scenario (group I: A, C, E; group II: B, D, F). Is that correct?

Please let me know if any issues or questions,
Best,
Lucie

@jessica-davis
Copy link
Contributor Author

Hi @LucieContamin, I think you are correct. Essentially there is a new stochastic_run/ run_grouping id for each sample and location combination that connects the age_group, horizon, and scenario sub-group (ACE and BDF). This means that when comparing the effects of the vaccine strategy within either the ACE or BDF scenarios (not across) and you can compare the trajectories that were run with the same random seed and initial conditions in our model (so the only change is the vaccine uptake in 2024/25). Is there a different way I should have done the run_grouping/ stochastic run numbering? Please let me know, and I'm happy to update the file if it makes things easier.

Thanks!
Jessica

@LucieContamin
Copy link
Collaborator

Hi @jessica-davis , Thank you for the confirmation and the additional information. It sounds all good to me!

Thanks again,
Lucie

@LucieContamin LucieContamin merged commit a09ddb0 into midas-network:master May 8, 2024
1 check passed
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.

None yet

2 participants