Speeds up aggregation of interval variables in input data where the interval sets for different combinations of id_cols can vary a lot. I will highlight the main changes in the code, a lot of the changes are just moving things around.
[X] Have you successfully run devtools::check() locally?
[X] Have you updated or added function (and vignette if applicable) documentation? Did you update the 'man' and 'NAMESPACE' files with devtools::document()?
[ ] Have you added in tests for the changes included in the PR?
[X] Do the changes follow the ihmeuw-demographicscode style?
[ ] Do the changes need to be immediately included in a new build of docker-base or docker-internal? If so follow directions in those repositories to rebuild and redeploy the images.
[ ] Do the changes require updates to other repositories which use this package? If yes, make the necessary updates in those repos, and consider integration tests for those repositories.
[ ] If this is a private package did you use Jenkins to rebuild the internal pkgdown site?
Describe changes
Speeds up aggregation of interval variables in input data where the interval sets for different combinations of id_cols can vary a lot. I will highlight the main changes in the code, a lot of the changes are just moving things around.
I made sure these timing comparisons stayed the same or sped up https://ihmeuw-demographics.github.io/hierarchyUtils/articles/agg_scale_performance.html#timing-comparison-1
And aggregated all population input data in under 1 minute
What issues are related
Fixes #60 Fixes #64
Related to #51
Checklist
Packages Repositories
ihmeuw-demographics
R packages?devtools::check()
locally?devtools::document()
?ihmeuw-demographics
code style?docker-base
ordocker-internal
? If so follow directions in those repositories to rebuild and redeploy the images.