NYCPlanning / db-equitable-development-tool

Data Repo for the equitable development tool (EDDT)
MIT License
0 stars 0 forks source link

Update rent_median.py #251

Closed AmandaDoyle closed 2 years ago

AmandaDoyle commented 2 years ago

Add field to output. Addresses issue #250

td928 commented 2 years ago

Hey @AmandaDoyle let me know when you have a chance to take a look at the new output I just added in the most recent commit. The work is a little more involved I want it to but a lot of functions build out currently have limitations in handling both median and count tokens interchangeably. This is something I will need to think about for future refactoring.

For results I think it passed the gut check (60 total columns). I changed the category from households_payingrent to units_payingrent to keep it consistent as a subcategory. The only question I had is that I made right decision or not to keep the pct, pct_moe out for this as well since it is the denominator. Thank you!