search

Data4Democracy / internal-displacement

Studying news events and internal displacement.
43 stars 27 forks source link

Add city and region to location schema #120

Closed alexanderrich closed 7 years ago

alexanderrich commented 7 years ago

Resolves issue #112 . Also updates country codes to 3 chars in model.py since 2 chars seemed inconsistent with the schema.

alexanderrich commented 7 years ago

Looking around the repository a bit more it, seems like a lot of work has been happening on the "interpreter" branch - is that the proper development branch to be pulling into?

domingohui commented 7 years ago

I think the updated schema is already merged to master. So I guess it's okay to merge this to master?

alexanderrich commented 7 years ago

I just closed the pull request and will make a new one on "interpreter". It looks like the sql schema has been much more integrated into the pipeline on that branch so I'll make sure that the province and city terms extracted by the interpreter are populated into the Location table.