Peter Johnson

Implement `shapeutil_visit_crossing_edge_pairs`

2

This PR implements `shapeutil_visit_crossing_edge_pairs` which is required to resolve https://github.com/golang/geo/issues/108. If this is something you'd be interested in merging then let me know. All the tests (so far) are passing,...

`CellUnionFromIntersection` calls `Normalize()`

2

Both the CPP and Go versions of `CellUnionFromIntersection` state that the intersection algo produces a result that is in sorted order. The go code calls `cu.Normalize()` anyway, what is the...

### Bug description Looking at [parameters.js](https://github.com/pelias/openaddresses/blob/master/lib/parameters.js) the help text seems to be out-of-date. Specifically it mentions `--admin-values` which seems to be a previously supported flag, and doesn't mention `parallel-count` or...

This PR adds tests for ARM architecture instances. Some of the floating-point math values differ between architectures, notably: - negative zero in some cases rather than positive zero - some...

### Bug description As discussed in https://github.com/pelias/parser/pull/195#pullrequestreview-3238761585 is seems the following classification is safe to remove in `classifier/scheme/street.js:120`: ```bash not: ['IntersectionClassification'] ``` ### Steps to reproduce as discussed in https://github.com/pelias/parser/pull/195...

This line should be rewritten in such a way that it accepts any input which is 'array-like' (ie. can be destructured with the spread operator) such that either an array...

As discussed in https://github.com/pelias/spatial/issues/106, the thread contains a lot of great performance analysis and instructions on tuning. closes https://github.com/pelias/spatial/issues/106

tune `module.shard.complexity`

10

Each geometry is split into 'shards', where each shard contains a maximum of n vertices. This process is used to ensure that PIP operations are fast over large polygons. The...

Currently the `merge` operation trips all the registered `triggers`, which ends up using a bunch of CPU to generate things like the `shard` entries when they could simply be copied.

`placetypes-spec-latest.json` missing `constituency`

1

Since https://github.com/whosonfirst/whosonfirst-placetypes/commit/5f5c4349599ce1e5401a04a83444466a2737de77 the `constituency` placetype has been removed from `placetypes-spec-latest.json` https://github.com/whosonfirst/whosonfirst-placetypes/blob/main/data/placetypes-spec-latest.json