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

Improvements: Roads + Boundaries + Landcover + Estimated Locality Populations #2013

Merged
merged 23 commits into from
Dec 2, 2021

Conversation

tgrigsby-sc
Copy link
Contributor

@tgrigsby-sc tgrigsby-sc commented Nov 20, 2021

These changes were visually inspected. Will follow up with test updates once we finish fixing the currently-broken integ tests.

nvkelso and others added 23 commits November 12, 2021 16:45
OSM data is very messy for these

Useful Overpas query:

```
/*
This has been generated by the overpass-turbo wizard.
The original search was:
“place=town and population=*”
*/
[out:json][timeout:250];
// gather results
(
  // query part for: “place=town and population=*”
  node["place"="city"]["population"](if:t["population"]<5000)({{bbox}});
  way["place"="city"]["population"](if:t["population"]<5000)({{bbox}});
  relation["place"="city"]["population"](if:t["population"]<5000)({{bbox}});
  node["place"="town"]["population"](if:t["population"]<5000)({{bbox}});
  way["place"="town"]["population"](if:t["population"]<5000)({{bbox}});
  relation["place"="town"]["population"](if:t["population"]<5000)({{bbox}});
);
// print results
out body;
>;
out skel qt;
```
And names from ferries early
…ndaries' into travisg/20211119-amalgam-build
@tgrigsby-sc tgrigsby-sc changed the title [WIP] Travisg/20211119 amalgam build Improvements: Roads + Boundaries + Landcover + Estimated Locality Populations Dec 1, 2021
@tgrigsby-sc tgrigsby-sc requested a review from nvkelso December 1, 2021 16:08
@nvkelso
Copy link
Member

nvkelso commented Dec 2, 2021

Replaces these 4 PRs:

@nvkelso
Copy link
Member

nvkelso commented Dec 2, 2021

I checked this visually and filed some followups. But looking amazing!

@nvkelso nvkelso merged commit 5dfc1d1 into master Dec 2, 2021
@nvkelso nvkelso deleted the travisg/20211119-amalgam-build branch December 2, 2021 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants