opensafely-core / opensafely-core/opencodelists
Refactor bnfdmd mapping scraping to use lxml
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 60
- Forks
- 16
- Avg merge
- 4d 12h
- Merged PRs (30d)
- 17
Description
60c6ce8 introduced a scraper for the NHSBSA website to get the latest BNF to dm+d mapping file.
It was heavily based on the OpenPrescribing scraper, which uses BeautifulSoup.
At the time, this introduced BeautifulSoup as a requirement for OpenCodelists.
This functionality could be reimplemented using lxml to remove this additional dependency.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing commit 60c6ce8 and the current NHSBSA BNF to dm+d mapping scraper. Compare its BeautifulSoup-based approach with OpenPrescribing's fetch_bnf_snomed_mapping.py, then identify the project's relevant scraper checks. Done means the mapping still works while BeautifulSoup is no longer an OpenCodelists dependency.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100