Peter Johnson
Peter Johnson
It's fairly common to find geometries with topology errors. A [GEOS TopologyException](https://cvvergara.github.io/GEOS/classgeos_1_1util_1_1TopologyException.html) is thrown due to the polygon rings overlapping and various other common errors. It's possible to leverage `GEOS`...
The list of spatial relationships can be very large (eg. `WITHIN(earth)`). Currently, there is a hard limit on how many are shown in the front-end (I believe it's 100). This...
The title is a bit of a misnomer... The idea is that we can use point-based geometries in the PIP engine so that they are returned in results when they...
The documentation is currently a bit thin. In order to reach a wider developer audience and attract outside contributors, it would be beneficial to add the following to the readme:...
When running `npm install` there are some warnings which could probably be easily fixed: ```bash npm WARN deprecated [email protected]: Use String.prototype.trim() instead npm WARN deprecated @hapi/[email protected]: Moved to 'npm install...
I was looking over the code today and noticed [this block](https://github.com/pelias/whosonfirst/blob/master/src/readStream.js#L59-L62) introduced in https://github.com/pelias/whosonfirst/commit/c500aaf3e77e6698a3f5966f2859e574e288e2dd which stores admin records in memory so they can later be used for computing hierarchies. It...
Some place names contain multiple consecutive spaces, it should be safe to squash these down to a single space: ``` "county": "Division No. 6", "county_gid": "whosonfirst:county:1158868963", ```
When interpolations are generated, they are always of type `address`, however it's possible for the API to return these results even when the layers are specified in a way that...
When running node `v19.9.0` a bunch of the tests fail due to the `Error.stack` format changing between node releases. I started hacking on `strip()` in `test/util.js` but it feels wrong,...
Updated `sed` to be portable between OSX & Linux I tried running this file in CI today and it failed due to the host OS being Linux: ``` sed: can't...