Skip to content

Commit

Permalink
Merge pull request #1411 from opensafely/steve/fix-python-docker-links
Browse files Browse the repository at this point in the history
Fix links to Python Docker image
  • Loading branch information
StevenMaude authored Dec 14, 2023
2 parents 756544d + 4268d34 commit 4c37c8d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion docs/actions-reusable.md
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ if __name__ == "__main__":
When developing a reusable action, just as when developing a scripted action, the action's dependencies are made available by the runtime; they are not made available by the action.

* The Python runtime is provided by [`python-docker`](https://github.com/opensafely-core/python-docker).
Its dependencies are in [*requirements.txt*](https://github.com/opensafely-core/python-docker/blob/main/requirements.txt).
Its dependencies are in [*requirements.txt*](https://github.com/opensafely-core/python-docker/blob/main/v1/requirements.txt).
In practice, this means that a Python action's *requirements.txt* is ignored.
* The R runtime is provided by [`r-docker`](https://github.com/opensafely-core/r-docker).
Its dependencies are in [*packages.csv*](https://github.com/opensafely-core/r-docker/blob/master/packages.csv).
Expand Down
4 changes: 2 additions & 2 deletions docs/actions-scripts.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ Scripted actions can read and write output files that are saved in the workspace

Outputs labelled `highly_sensitive` will not be visible to researchers. This is a [deliberate design feature of OpenSAFELY](https://www.opensafely.org/about/), intended to reduce the risk of disclosure of sensitive information. Outputs should **always** be classed as `highly_sensitive` if they are:

- Pseudonymised patient-level outputs derived from queries run against Level 1 and 2 data, i.e., a specific study dataset generated by a [dataset_definition](https://docs.opensafely.org/ehrql/).
- Pseudonymised patient-level outputs derived from queries run against Level 1 and 2 data, i.e., a specific study dataset generated by a [dataset definition](ehrql/index.md).
- Pseudonymised patient-level intermediate outputs for a study derived from queries run against Level 3 data which output pseudonymised patient-level data i.e., a processed study dataset with certain filters/formatting applied.

These types of outputs are considered potentially highly-disclosive, should not be pushed to Level 4, and are never intended for publishing outside the secure environment.
Expand Down Expand Up @@ -168,7 +168,7 @@ The `opensafely` command line software will now automatically use this Stata lic

### Python

The Docker image provided is Python 3.8, with [this list of packages installed](https://github.com/opensafely-core/python-docker/blob/main/requirements.txt).
The Docker image provided is Python 3.8, with [this list of packages installed](https://github.com/opensafely-core/python-docker/blob/main/v1/requirements.txt).

### R

Expand Down
2 changes: 1 addition & 1 deletion docs/requesting-libraries.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ issue](https://github.com/opensafely-core/stata-docker/issues) to request the pa

## Python packages

First check that the package is not already available, by [viewing all installed packages and their versions for the OpenSAFELY Python image](https://github.com/opensafely-core/python-docker/blob/main/requirements.in). If your package isn't there then [open an
First check that the package is not already available, by [viewing all installed packages and their versions for the OpenSAFELY Python image](https://github.com/opensafely-core/python-docker/blob/main/v1/requirements.txt). If your package isn't there then [open an
issue](https://github.com/opensafely-core/python-docker/issues) to request it be added. Include the following information:

* A link to information about the package;
Expand Down

0 comments on commit 4c37c8d

Please sign in to comment.