Peter Johnson

Results 1014 comments of Peter Johnson

@geopamplona I can see that *maybe* this line is causing the problem https://github.com/pelias/openstreetmap/blob/master/stream/pbf.js#L48 ...but the edge case I'm considering would only be under these conditions: - you had multiple values...

Interestingly, the name is correct when asking for a different language as per `/v1/place?lang=ko&ids=geonames:neighbourhood:2188922` So I guess the error must lie with the translation code in API.

This doesn't seem to be in the `changeLanguage` middleware, because even if it was doing a `placeholder` query, there is no matching id in WOF: ``` placeholder://parser/findbyid?ids=2188922 {} ```

The [Eiffel Tower](https://www.openstreetmap.org/way/5013364) is unfortunately also tagged `man_made=tower`.

I've not seen that `frequency` tag before, maybe we can just exclude all records with that?

http://www.openstreetmap.org/way/10525375 **[edit]** wontfix re: adding `highway` tags for the same reason as https://github.com/pelias/openstreetmap/issues/28 plus this is not a POI.

consider blacklisting `public_transport:stop_area` http://www.openstreetmap.org/node/2478086894 **[edit]** it seems best to completely remove `public_transport` because of duplicates, see https://gist.github.com/missinglink/b7757d98d034f1441b74

http://www.openstreetmap.org/node/2907810083 **[edit]** same as above, there are a lot of duplicates for `railway:*` https://gist.github.com/missinglink/e7f1de91670e7173792c

http://www.openstreetmap.org/node/2478086894 **[edit]** same as `public_transport:stop_area` above

hi @razafinr crazy timing, I was just looking in to this issue earlier today, I've been cutting some data from a New York City extract to have a look what...