lxml installation

Open
#86 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
35/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
docker, python
Domain
build-system, devops

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.

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

Dominant language
Python
Stars
0
Forks
3
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

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.

More from opensafely-core/python-docker

All issues in opensafely-core/python-docker

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.