opensafely-core / opensafely-core/python-docker
lxml installation
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 0
- Forks
- 3
- PR merge metrics
- No merged PRs in 30d
Description
Hi @data-team,
I need lxml to be installed since I ran into an error in reading .html files. The error is lxml not found, please install it, and you can find it in the hosp_inf_pred workspace and it appears when I run generate_notebook_aggregate_cox_hrs_betas job. It would also be needed for generate_notebook_aggregate_cox_hrs_antibiotics and generate_notebook_aggregate_cox_cis jobs. Without running these jobs I would need to request for hundreds of single .html files to be released, which is not preferred. Further information about lxml can be found in this link: https://lxml.de/.
Thanks,
Ali
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
Review the python-docker image configuration to find where Python dependencies are declared. Use the hosp_inf_pred workspace and run generate_notebook_aggregate_cox_hrs_betas, generate_notebook_aggregate_cox_hrs_antibiotics, and generate_notebook_aggregate_cox_cis to verify that lxml is available and the missing-module error is resolved.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- docker, python
- Domain
- build-system, devops
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100