-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
Point in Polygon related test tweaks 2017-09-26 #434
Conversation
test_cases/reverse_coarse.json
Outdated
"region": "Puerto Rico", | ||
"dependency": "Puerto Rico", | ||
"country": "United States" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removing country here and not replacing it with anything else, like empire, feels a little unsettling
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah agreed. @trescube please opine.
looks good to me, I am not adding a review because I'm not sure about the |
efbd87a
to
e40a3e1
Compare
This likely changed due to differences in admin lookup strategies
Ok this has all been updated and the Puerto Rico case is no longer being handled here. See #437 for the handling of that issue. |
This PR updates our acceptance tests for PIP related changes.
There is one case of a previously working test failing because the new admin lookup strategy has exposed a broken WOF hierarchy, as well as some name changes. One other test is changed to reflect an updated hierarchy, which may have also been because of our admin lookup changes.