Skip to content

Conversation

@missinglink
Copy link
Member

It seems that libpostal gets confused with airports, returning the label suburb:

curl --get http://localhost:4400/parse \
  --data-urlencode 'address=john f kennedy international airport'

[{"label":"suburb","value":"john f kennedy international airport"}]

However it returns the correct label for a neighbourhood:

curl --get http://localhost:4400/parse \
  --data-urlencode 'address=soho'

[{"label":"suburb","value":"soho"}]

And also in the case where the text refers to either an airport or a neighbourhood:

curl --get http://localhost:4400/parse \
  --data-urlencode 'address=tegel'

[{"label":"suburb","value":"tegel"}]

I think the best course of action here is to check for any parse containing a single label of type "suburb" and then discarding it.
While this certainly will discard some valid neighbourhood parses, they should be adequately handled by the fallback parsing.

@missinglink missinglink requested a review from orangejulius June 21, 2022 14:47
@missinglink missinglink force-pushed the libpostal-suburb-fix branch from 4a7ecf6 to ea065bb Compare June 21, 2022 14:51
@missinglink missinglink force-pushed the libpostal-suburb-fix branch from ea065bb to 293ef5c Compare June 21, 2022 14:57
@missinglink
Copy link
Member Author

falling back to the pelias/parser in these cases seems preferable:

Screenshot 2022-06-21 at 17 00 29

Screenshot 2022-06-21 at 17 00 50

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