Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

import population data from 'qs:pop' #106

Closed
missinglink opened this issue Jun 30, 2016 · 6 comments
Closed

import population data from 'qs:pop' #106

missinglink opened this issue Jun 30, 2016 · 6 comments
Assignees
Labels

Comments

@missinglink
Copy link
Member

missinglink commented Jun 30, 2016

the following record has a property called qs:pop which contains population info, that property may be new and is not currently being checked during import.

@orangejulius
Copy link
Member

Sounds good to me. @nvkelso, is this a field we should be using?

Right now we first look at gn:population and then zs:pop10. Where should qs:pop go in that ordering?

@trescube
Copy link
Contributor

Or have a general mz:population field would be ideal so we don't have to look in one place for neighbourhoods and another for localities.

@nvkelso
Copy link

nvkelso commented Jun 30, 2016

mz:population is my preference

On Thu, Jun 30, 2016 at 6:38 AM, Stephen K Hess notifications@github.com
wrote:

Or have a general mz:population field would be ideal so we don't have to
look in one place for neighbourhoods and another for localities.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#106 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AA0EO70Xheoww__l3KRe0XjNPdMXF6SGks5qQ8bDgaJpZM4JCJsh
.

@orangejulius
Copy link
Member

Awesome, I suspect we'll want to handle qs:pop ourselves in the meantime (we don't want to rush you to release it when the change on our side is relatively small). Any suggestion for an ordering of the 3 variables?

@trescube
Copy link
Contributor

I believe that zs:pop10 only applies to neighbourhoods.

@dianashk
Copy link
Contributor

Update the code to import all 3: gn:population, qs:pop, zs:pop10.
Will switch to mz:population when that becomes available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants