Closed nvkelso closed 9 years ago
I'm seeing some more POIs, earlier (good), but we need more and earlier – at least in San Francisco. Let's try adjusting the area thresholds &/or the zoom offsets in the earlier #222 PR.
Here's some stuff I've pulled out. Note that the text is all busy because I've enabled a text label for every single point, so you can see everything that's in the data. I've also disabled all the landuse areas, because I haven't merged the logic for those yet.
@nvkelso, it would be really helpful if you could give me some feedback on the images above and whether stuff is coming in too early, too late, or the wrong stuff is visible, etc... Just bear in mind that it doesn't include any of the landuse area centroids - I'm working on that and hopefully will have some screenshots of that soon.
Generally this is a huge improvement, thank you!
Should we roll this out incrementally? (What you have already seems good enough to move to prod if you think the code is good enough.)
I've marked up the images. Here's the summary, images down farther.
Please make these types of features visible earlier, as annotated on the images:
We should probably show screenshots of New York and London, too.
Latest screenshots, based on the work in #265.
There's a high density of POIs around Golden Gate Park, which I think would be best addressed by #266 rather than further tweaking.
There's still some duped labels in here, which it would be good to get rid of in #267.
Most of the subway stations kick in all together at this zoom - possibly they're all point features. It would be nice to differentiate on "importance", perhaps by pulling the number of different lines it's part of out of the relation?
@nvkelso what do you think? OK to go, and we'll come back to fix #266, #267 and any additional tweaks in another round?
Generally looking fantastic! You've added a lot of clarity to the map :)
Regarding z13 in NYC and all the subway stations, let's track that using #268 as a separate problem. Are you suggesting we should lower the default layer to zoom 14 instead of 13 (you propose 13, it is a blanket 12 now) for those, though?
I'm still not seeing big churches? (they aren't visible now, so not a regression.) Is that because there isn't any landuse polygon with those tags? Should we open a separate issue to adopt this logic for POIs vis-a-vis building polygons?
It looks like you're using the area to calculate the visible airports, can you show what that looks like at zooms 9 and 10? We've usually had too many of those show up, I want to make sure we're seeing less after this change. Alternatively we could load this into dev and have a look around.
It looks like you're using the area to calculate the visible airports, can you show what that looks like at zooms 9 and 10? We've usually had too many of those show up, I want to make sure we're seeing less after this change.
No airports... we had too many, now do we have too few? Do we want them at zoom 9? It should be easy to pull them up a zoom, if that's what you want.
Here we've got two: SF International (seems appropriate) but also Oakland, which is actually a larger airport in terms of area. If we need to distinguish between Oakland and SFO, then we probably need to bring in another dataset such as passenger numbers.
Again, no airports. Although note that this extract is much smaller than the Bay Area one, so might be missing a few that didn't make the cut.
This zoom has Newark and JFK, which seems appropriate.
Here's a list of the top 20 airports on my local machine and the zoom they currently cut in at. Let me know if these can be dropped by a zoom or half or something.
min_zoom | name
----------+-------------------------------------------------
8 | Andrews AFB
9 | John F. Kennedy International Airport
9 | Sacramento International Airport
9 | McClellan Airfield
9 | Metropolitan Oakland International Airport
9 | Newark Liberty International Airport
9 | San Francisco International Airport
9 | Gnoss Field Airport
10 | Norman Y. Mineta San Jose International Airport
10 | Fayetteville Regional Airport
10 |
10 | Charles M. Schulz - Sonoma County Airport
10 | Moffett Federal Airfield
10 | LaGuardia Airport
10 | Teterboro Airport
10 | Westchester County Airport
10 | Stockton Metropolitan Airport
10 | airport
10 | Farmingdale Airport
10 | Eagle County Regional Airport
After 918ba48 in #265 to pull places of worship up a few zooms:
I suspect the stylesheet has an area filter that needs adjusting at zooms 9/10 so more tile data renders, I'll investigate that.
Churches fixed in #265, so when that's merged this is done.
This is so awesome :)
I've updated the landuse area filters in the scene file. #213 covers porting some of that logic server side... when you disable those filters now a crazy number of label_position
features show on the map, but that's always been the case and is already covered by that earlier issue. This is ready for prod.
Today we show all features from a kind class at the same initial zoom, but this results in a boring and busy, poorly balanced map.
For instance, we start showing hospitals first, but we end up showing a mix a large & important hospitals and small and less important clinics at the same initial zoom. Instead only the large & important hospitals should be shown at first, and then the smaller, less important clinics should be added progressively in later zooms.
Many POIs are technically labeling landuse AOIs and building featuers and we can leverage the parent's area to throttle the POIs zoom (first adopting the same landuse / building area thresholds and then refining them to be better for POIs – for instance, we see some delta between landuse area, landuse-labels area, and building area for calculating label visibility today.