pelias / openaddresses

Pelias import pipeline for OpenAddresses.
MIT License
52 stars 44 forks source link

analysis: append ordinal suffix to numeric street names #508

Closed missinglink closed 2 years ago

missinglink commented 2 years ago

following on from https://github.com/pelias/openaddresses/pull/477 et al, this is something we were missing which annoyed me. it was a relatively simple task to add missing street ordinals to numeric street names.

Screenshot 2022-06-13 at 12 45 26
missinglink commented 2 years ago

This looks good, I tested it on a global build and it had several positive changes to our test suite, particularly in grid cities like NYC.

Screenshot 2022-06-14 at 12 36 25 Screenshot 2022-06-14 at 12 38 10 Screenshot 2022-06-14 at 12 38 15 Screenshot 2022-06-14 at 12 38 28 Screenshot 2022-06-14 at 12 38 41
orangejulius commented 2 years ago

Yeah, this looks great. Very good change.