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

Rethink mappings to "basic cohort attributes" #128

Open
beckyjackson opened this issue Nov 9, 2020 · 1 comment
Open

Rethink mappings to "basic cohort attributes" #128

beckyjackson opened this issue Nov 9, 2020 · 1 comment
Labels

Comments

@beckyjackson
Copy link
Collaborator

GECKO terms like "aims and objectives" and "location" (e.g., the location in which the cohort population is) seem like they are cohort metadata items, rather than terms that should be mapped to individual data dictionary items.

I think this is true for all the terms in the "basic cohort attributes" branch.

I'm wondering if we should add these to the Metadata table as True or False fields? Or even fields that can be filled in? It would make sense that all cohorts have aims and objectives, and all cohorts have a location or locations in which their population is based. None of this is currently captured.

@beckyjackson
Copy link
Collaborator Author

beckyjackson commented Nov 9, 2020

@rosibaj this may result in a change (if we decide to move them around) to how these variables are tracked in the cohort JSON so let's keep in touch about that.

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

No branches or pull requests

1 participant