Repository navigation
fix: suggest Canadian houses whose source omits the street direction, and reject the opposite one - #515
Conversation
Merging this PR will not alter performance
Comparing Footnotes
|
…tion, and never take the opposite one
0773029 to
97e8866
Compare
Speed, full index: no slowdown over 15%DetailsSearchAddressTime in ms over 1993 cases, each the fastest of 3 requests.
SuggestAddressTime in ms over 2136 checks, each the fastest of 3 requests.
ReverseGeocodeTime in ms over 1990 cases, each the fastest of 3 requests.
Fails when the total, p50, p90 or p99 of any table is more than 15% slower. Runs of one binary differ by up to 5%; the max is one request and only reported. Base |
Gold check, full index: no regressions
Pass rate by countryPassing checks out of those checked, baseline → candidate, for the 1 expected country where any RPC changed. A case that expects several countries has its own row. Counts are checks, not independent locations.
Runs
SuggestAddress
Newly passing (verify before relabelling) (8)
|
Summary
Canadian mailing forms such as
31 THATCHER DR E, MOOSE JAW, SK S6J 1L8now get a house suggestion when the source writes the street without its direction (Thatcher Drive). Search already handled these; suggestions did not. The existing search retry also had a defect: it could answer a road with the opposite direction, so870 6e Rue Estreturned6e Rue Oas a good match. Both now reject a road whose direction the query does not state.Changes
ca::other_street_directionrejects a road that has a direction the query's street part lacks.Ouest,OandWcount as the same direction. A source that omits the direction is not a conflict. Applied to both the search and suggest retries.Thistle Bend Drive) with search and suggest checks, including wrong or missing postcode, missing number or suffix, opposite direction and a US country filter, plus a unit test for the direction check.Review focus
Check
other_street_directionfor how it treats the query's street part (text before the first comma) and the road's direction words.