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

Defunct Tenderloin neighbourhood displays in NYC #1869

Closed
nvkelso opened this issue Apr 12, 2019 · 4 comments
Closed

Defunct Tenderloin neighbourhood displays in NYC #1869

nvkelso opened this issue Apr 12, 2019 · 4 comments
Assignees
Milestone

Comments

@nvkelso
Copy link
Member

nvkelso commented Apr 12, 2019

Related to #1808.

Screen Shot 2019-04-12 at 13 46 04

@nvkelso nvkelso added this to the v1.8.0 milestone Apr 12, 2019
@nvkelso
Copy link
Member Author

nvkelso commented Apr 12, 2019

By updating the WOF build via #1808 I initially thought we'd remove the bad Tenderloin label, but it's been deprecated for such a long time there might be an additional logic error related to processing mz:is_current.

@zerebubuth
Copy link
Member

I loaded up the WOF data locally to check, and got this result:

# select is_visible from wof_neighbourhood where wof_id = 85869725;
 is_visible 
------------
 f
(1 row)

If is_visible is false, that should mean it gets selected by neither the query code in vector-datasource nor the query code in raw-tiles. However, to be sure, we'll have to wait for the latest build to finish.

Although the property edtf:deprecated: 2016-08-16 suggests deprecation in 2016, it looks like the WOF record was last modified on 2018-05-25. The WOF pgdump we store on S3 says Last-Modified: Wed, 14 Mar 2018, which is before the WOF record was last changed. I'm not sure how to check the previous versions of a WOF record, but it seems possible that the deprecated date was modified after the last static dump we created.

@nvkelso
Copy link
Member Author

nvkelso commented Apr 15, 2019

Hard to say, even looking at the blame:

Let's wait for a new build before researching anymore :)

@nvkelso
Copy link
Member Author

nvkelso commented Apr 22, 2019

No longer in dev build, closing.

@nvkelso nvkelso closed this as completed Apr 22, 2019
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

No branches or pull requests

2 participants