BasisResearch / cities

Home of Basis development for the 2023 TOP Sprint
MIT License
6 stars 0 forks source link

Adding climate and economic justice variables, and age composition (counties) #102

Closed Niklewa closed 8 months ago

Niklewa commented 8 months ago

In this PR, I have worked on variables from the areas of climate hazards, housing and energy burdens, health, and age distributions.

The major issue is that in the upcoming PRs, which have not yet been merged, there are changes to the files containing functions intended for cleaning in this PR. Firstly, it needs to be pulled from branches that are to be merged, and then the cleaning pipeline should function correctly (I do not want to modify those functions in this version to avoid conflicts).

Niklewa commented 8 months ago

I have pulled all the most recent changes from the target branch. I have run the cleaning pipeline for all the files because there were some counties lost along the way (just a couple of them). I have cleaned the health variable, but ultimately, I have not introduced it since it would result in losing around 15 counties; I'm not sure if it is worth it.

Some tests are failing, and I have checked them. I believe they fail because the freshest version of our dataset is not compatible with the trained causal framework. They became an issue after running the entire pipeline.