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

PSI-PROF: Update RSV submission #16

Merged
merged 1 commit into from
Jan 8, 2024
Merged

Conversation

jturtle
Copy link
Contributor

@jturtle jturtle commented Dec 29, 2023

  • We found that the final calibration step was not always working as intended. The issue has been fixed and results updated for this commit.

- We found that the final calibration step was not always working as intended.  The issue has been fixed and results updated for this commit.
Copy link

[1] "Run validation on file: 2023-11-12-PSI-PROF.gz.parquet"

Columns:

No errors or warnings found on the column names and numbers

Scenarios:

No errors or warnings found on scenario name and scenario id columns

Origin Date Column:

No errors or warnings found on the column 'origin_date'

Value and Type Columns:

No errors or warnings found on Value and Type columns

Target Columns:

🟡 Warning 602: No value found associated with the targets: inc hosp (optional), cum hosp (optional); output_type: quantile.
🟡 Warning 602: No value found associated with the targets: peak size hosp (optional); output_type: quantile.
🟡 Warning 602: No value found associated with the targets: peak time hosp (optional); output_type: cdf.

Locations:

No errors or warnings found on Location

Sample:

No errors or warnings found on Sample

Quantiles:

No errors or warnings found on quantiles values and format

Age Group:

No errors or warnings found on Age_group

@LucieContamin
Copy link
Contributor

Thank you for the update,
I just wanted to verify I understand correctly the "sample" run_groupingand stochastic_run columns. It seems that each model run has a different run_grouping set grouped by age_group, horizon, scenario_id (model run independent) and the runs are not stochastic (no stochasticity), is that correct?

Please feel free to let me know if any questions or issues,
Best,

Lucie

@LucieContamin LucieContamin merged commit 7a9f865 into midas-network:main Jan 8, 2024
1 check passed
@jturtle
Copy link
Contributor Author

jturtle commented Jan 8, 2024 via email

@LucieContamin
Copy link
Contributor

Hi Jamie,
Thanks for the reply,
Following your answer, the two columns seem to be all correct to me. Thanks again

Best, Lucie

@jturtle
Copy link
Contributor Author

jturtle commented Jan 8, 2024 via email

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