-
Notifications
You must be signed in to change notification settings - Fork 40
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
example_calm_repop not running #169
Comments
Repop works once I skip the summarize.repop module in the settings file. |
Hi @jawadmhoque, are you running the example as is, or did you update any of the settings? I noticed the settings.yaml file for example_calm_pop is configured to resume from summarize step: |
Hey @binnympaul . I am running the example as is. I tried setting the resume_after step to blank and it fails at the repop_setup_data_structures step with a Key Error on households. Here is the command line outputs:
But running it with the pipeline from the example_calm output and skipping the summarize step still works. As far as I can understand from here , repop mode starts from the last step of the regular run, which is the summarize step. So shouldn't it be specified like that? |
Can confirm I'm getting the same error as @jawadmhoque. And also that the workaround of skipping |
@jawadmhoque , @conrad-mac : Thanks for confirming the issue. You are right, the repop run must be configured to resume from the I reviewed the code and settings, and looks like the list of geographies should list only the lowest geography among the sub-seed geographies. This is because the controls are only specified at the lowest geographic level for a repop run and other sub-seed geographies are not available for summaries. The The workaround of skipping |
Confirming that editing the geographic setting to have the lowest geography among the sub-seeds work with the |
Updated repop settings lead to null values in the second level geographies. Created new issue #170 . |
The repop synthesis fails at the summarize step (line 60 in summarize_geography). Not all the unique hh ids in the zone_weights dataframe is there in the incidence_df.
populationsim release:
Error message:
The text was updated successfully, but these errors were encountered: