pelias / pelias/parser

Recognise UK postcodes with space in between

Open
#135 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
JavaScript
Stars
64
Forks
26
PR merge metrics
No merged PRs in 30d

Description

Regex works properly. However, it's further upstream the space in between breaks the postal code into 2 sections. Also happens for NL postcodes. Examples:

  • SW11 6NU
  • BA14 7LY
  • EC1N 2NS

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing the parser's upstream splitting or tokenization path, since the issue says the regex already works and the space separates the postcode into two sections. Reproduce the behavior with SW11 6NU, BA14 7LY, EC1N 2NS, and the mentioned NL postcode cases. Done means spaced postcodes remain intact for downstream classification.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
search
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.