search

NeighborhoodInfoDC / BridgePk

11th Street Bridge Park project
0 stars 0 forks source link

Data #4 #8

Closed sdiby closed 6 years ago

sdiby commented 7 years ago

@ptatian I've added new ACS tabs to BP program at city, ward, and bridgepark levels.

sdiby commented 7 years ago

@rpitingolo @ptatian Rob, this program is also ready for your review.

rpitingolo commented 7 years ago

@sdiby I'll review.

rpitingolo commented 7 years ago

@sdiby please check where you have GrossRent: in the variable lists. I think you need to change it to include the _&year.

sdiby commented 7 years ago

@rpitingolo I fixed GrossRent vars--program is ready for your review.

rpitingolo commented 7 years ago

@sdiby thanks. Have you done the final batch submit for the ACS data? I think that needs to happen before I can test these fully.

sdiby commented 7 years ago

@rpitingolo sorry about that--thanks for catching that. Final batch submits for dc, md, and va ACS data are now complete.

rpitingolo commented 7 years ago

@sdiby it looks like RentCostBurdenDenom_2011_15 and OwnerCostBurdenDenom_2011_15 are missing from all of the ACS summary files. I also think for the city-data we should switch to have it use ACS.acs_2011_15_dc_sum_regcnt_regcnt since that's the actually county-level data, not built up from tracts.

sdiby commented 7 years ago

@rpitingolo program runs without errors on my machine. Thanks again for helping with the ACS macro errors.

If this looks okay, feel free to review and merge.

ptatian commented 7 years ago

@rpitingolo It seems like work on this issue is complete? If so, would you please merge the pull request and close issue #4? Thank you.

rpitingolo commented 7 years ago

@ptatian I'm double checking this one and will close/merge once confirmed.

ptatian commented 7 years ago

@rpitingolo Checking in again on the status of this issue.

ptatian commented 6 years ago

@rpitingolo @sdiby Hi folks. This pull request, and associated issue #4, remains open but it seems like the work was completed long ago. Is there any reason why it should not be merged and closed?

rpitingolo commented 6 years ago

@ptatian no. I'll merge and close this one.