Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Case control and measures pages haven't been updated to use ehrQL #1492

Open
StevenMaude opened this issue Apr 23, 2024 · 2 comments
Open

Case control and measures pages haven't been updated to use ehrQL #1492

StevenMaude opened this issue Apr 23, 2024 · 2 comments
Assignees
Labels
improve content Improve existing documentation content

Comments

@StevenMaude
Copy link
Contributor

Examples:

Our *project.yaml* now includes the following action:
```yaml
# ...
actions:
# ...
extract_controls:
run: cohortextractor:latest generate_cohort --study-definition study_definition_controls --output-format csv.gz
needs: [matching]
outputs:
highly_sensitive:
cohort: output/input_controls.csv.gz

### Define a study definition
The `study_definition.py` script for the example above is:
```py
from cohortextractor import StudyDefinition, Measure, patients
study = StudyDefinition(
# Configure the expectations framework
default_expectations={
"date": {"earliest": "2020-01-01", "latest": "today"},
"rate": "exponential_increase",
"incidence": 0.2,
},

@StevenMaude StevenMaude added the improve content Improve existing documentation content label Apr 23, 2024
@lucyb
Copy link
Contributor

lucyb commented Jul 24, 2024

I've spoken to Brian and someone in the research team will be able to update this code.

@wjchulme
Copy link
Contributor

Regarding measures. There are currently two pages covering measures in the documentation.

Anything under legacy doesn't show up in searches. So the page is harmless and can stay there. As far as I can see there's nothing that needs to be updated for the new measures ehrQL page, so we're done. I don't know which commit(s) fixed this between this issue being opened and now, so I won't link to it.

All that remains for this issue is the case-control docs, which @alexwalkerepi has agreed to take a look at.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improve content Improve existing documentation content
Projects
None yet
Development

No branches or pull requests

4 participants