Initialize_city.py should add best-guess latitude & longitude values to city config

insight-lane / crash-model

Build a crash prediction modeling application that leverages multiple data sources to generate a set of dynamic predictions we can use to identify potential trouble spots and direct timely safety interventions.

https://insightlane.org

MIT License

113 stars 40 forks source link

Initialize_city.py should add best-guess latitude & longitude values to city config #144

Closed terryf82 closed 5 years ago

terryf82 commented 5 years ago

@j-t-t here's the change:

https://github.com/Data4Democracy/crash-model/pull/149

I was going around in circles trying to get a test to work (see my message in Slack) so it's not in this PR. Happy to have another go pending your response but for now the minor change made is here if you want to review it, thanks.

terryf82 commented 5 years ago

Updated PR now includes basic test of initialize_city functionality -

https://github.com/Data4Democracy/crash-model/pull/149