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

senegal #138

Merged
merged 4 commits into from
May 19, 2023
Merged

senegal #138

merged 4 commits into from
May 19, 2023

Conversation

abhidg
Copy link
Contributor

@abhidg abhidg commented May 12, 2023

  • parsers: add ccp-senegal
  • schemas: allow date and start_date to include time
  • schemas(observation): add loss of smell or taste

adtl isaric/parsers/ccp-senegal.toml ../isaric-data/ccp-senegal/CVRZVOV_GLOBAL_Changes.csv

table valid total percentage_valid
subject 200 200 100.000000%
visit 200 200 100.000000%
observation 25011 25019 99.968024%

observation

  • 6: data must contain ['phase', 'date', 'name'] properties
  • 2: data must be valid exactly by one definition (0 matches found)

@pipliggins
Copy link
Collaborator

All the drugs administered (antivirals, antibiotics, steroids etc) appear to be stored in tmt_pdhosp_nom{n} variables (of which there are 20ish in the data). Any thoughts about how to include these? We'll have to work backwards from finding names and inferring that antivirals etc. were administered I think.

It's free text data, but appears to have been somewhat cleaned, it's all in caps so as a stop gap we could for example search for 'HYDROCORTISONE' in those fields, and if present then treatment_corticosteroid will be True. It's a long way from catching everything but prior to getting NLP in there that might be as good as we can do.

@abhidg
Copy link
Contributor Author

abhidg commented May 12, 2023

Will have a think, best to programmatically generate these mappings somehow. There are also some names in French and spelling errors.

@abhidg abhidg marked this pull request as draft May 12, 2023 17:04
Copy link
Collaborator

@ekamau ekamau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

isaric/parsers/ccp-senegal.toml Show resolved Hide resolved
schemas/dev/observation.schema.json Show resolved Hide resolved
isaric/parsers/ccp-senegal.toml Outdated Show resolved Hide resolved
isaric/parsers/ccp-senegal.toml Show resolved Hide resolved
isaric/parsers/ccp-senegal.toml Outdated Show resolved Hide resolved
isaric/parsers/ccp-senegal.toml Outdated Show resolved Hide resolved
isaric/parsers/ccp-senegal.toml Outdated Show resolved Hide resolved
isaric/parsers/ccp-senegal.toml Outdated Show resolved Hide resolved
@abhidg abhidg marked this pull request as ready for review May 19, 2023 10:43
@abhidg abhidg merged commit 7de38c3 into main May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants