311-data icon indicating copy to clipboard operation
311-data copied to clipboard

Updated population density of the LA city neighborhood councils (NCs).

Open priyakalyan opened this issue 2 years ago • 5 comments

Fixes #1229

  • [ ] Up to date with dev branch
  • [ ] Branch name follows guidelines
  • [ ] All PR Status checks are successful
  • [ ] Peer reviewed and approved

Any questions? See the getting started guide

priyakalyan avatar Sep 02 '22 18:09 priyakalyan

Sorry it took me a while to get to this! I took a first look at the first notebook. I will look again after you resolve the comments.

BTW, I'm using ReviewNB to review your notebooks. You might need to sign up for it, but it's free and works pretty well!

nichhk avatar Sep 21 '22 03:09 nichhk

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Hey @priyakalyan , not sure if you got notified for it, but I left some more comments here: https://app.reviewnb.com/hackforla/311-data/blob/NC-population/analytics%2Fpopulation%2Fnc_pop_recent.ipynb/discussion. I will take a look at the other notebook now.

nichhk avatar Oct 12 '22 01:10 nichhk

The comparison of the two methods looks good!

Note: I would say that for simpler analyses like this, we don't need to use Jupyter/Python. I think everything there could have been done in Google Sheets.

nichhk avatar Oct 12 '22 01:10 nichhk

Thank you Nich! I will keep this in mind.

priyakalyan avatar Oct 25 '22 16:10 priyakalyan

Thanks Nich for reviewing this PR!!!

priyakalyan avatar Nov 15 '22 16:11 priyakalyan