Skip to content
This repository was archived by the owner on Apr 21, 2026. It is now read-only.

feat: add support for ISARIC - #730

Merged
bloodearnest merged 3 commits into
mainfrom
isaric
Feb 7, 2022
Merged

bloodearnest merged 3 commits into
mainfrom
isaric

Conversation

@bloodearnest

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread cohortextractor/tpp_backend.py Outdated
Comment thread cohortextractor/tpp_backend.py
@bloodearnest
bloodearnest force-pushed the isaric branch 5 times, most recently from daee21c to 9780bf2 Compare February 3, 2022 12:22
@bloodearnest bloodearnest changed the title wip feat: add support for ISARIC Feb 3, 2022
@bloodearnest
bloodearnest marked this pull request as ready for review February 3, 2022 12:23
@bloodearnest
bloodearnest force-pushed the isaric branch 2 times, most recently from 87a3a13 to 6617301 Compare February 3, 2022 12:52
@bloodearnest
bloodearnest force-pushed the isaric branch 4 times, most recently from 536dc86 to 9071333 Compare February 7, 2022 13:43
The ISARIC dateset is very wide, with 300+ columns, all encoded as
strings. So, we include metadata generated from the TPP db to type each
column, as well as do some type conversion in the SQL itself.
@bloodearnest
bloodearnest merged commit 1d7416d into main Feb 7, 2022
@bloodearnest
bloodearnest deleted the isaric branch February 7, 2022 22:14
@iaindillingham iaindillingham mentioned this pull request Apr 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants