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

Added values 'Tulsa LIBR-1000', 'MIND', 'BEAM', 'MARS-WI', 'POINTER',… #104

Merged
merged 4 commits into from
Sep 5, 2024

Conversation

MasterPastures
Copy link
Contributor

… and 'ADRC' to cohort.csv and in valid values column

Per Robin French's request, values used detailed here: https://docs.google.com/spreadsheets/d/1uUpFVkjJ8ZW1RlZQ0nBB0WyayKw7cZdz/edit?gid=231099282#gid=231099282

Copy link

github-actions bot commented Sep 5, 2024

Test Suite Report

Template Generation

The following manifest templates had at least one dependency with
changes detected:

template result link
IndividualHumanMetadataTemplate template link

🚧 Manifest Validation - COMING SOON

Copy link
Contributor

@avanlinden avanlinden 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! Thanks for copying the definitions in, we're going to need to keep an eye out for contributors mixing up the 'MARS' cohort (from Rush, Minority Aging Research Study) and 'MARS-WI' (microbiome from Wisconsin, I guess).

@avanlinden avanlinden merged commit 8482222 into main Sep 5, 2024
@avanlinden avanlinden deleted the MasterPastures-patch-2 branch September 5, 2024 21:37
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.

2 participants