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

Last updates before major release #181

Merged
merged 15 commits into from
May 13, 2024
Merged

Last updates before major release #181

merged 15 commits into from
May 13, 2024

Conversation

lopierra
Copy link
Member

@lopierra lopierra commented May 3, 2024

Hi @madanucd - I made the updates to Study/Dataset (and other components) that we've been discussing over the past few weeks - see commit messages. Let me know if you notice anything I missed.

Also, would you mind generating the schema and pydantic to make sure we don't have any errors in the YAMLs? I realized I haven't actually installed LinkML on my new laptop, but I don't want to hold up the release. Thank you!

@lopierra lopierra requested a review from madanucd May 3, 2024 21:43
@twhetzel
Copy link

twhetzel commented May 3, 2024

@lopierra how soon do you need this reviewed?

@lopierra
Copy link
Member Author

lopierra commented May 3, 2024

Sometime next week, if possible? It would be good to have a major release of the LinkML model to coincide with our next release of the Data Hub

@madanucd
Copy link
Contributor

@lopierra
Participant and Sample IDs: Is it advisable to incorporate both External and Global IDs for participants and samples within the Biospecimen, DataFile, and Participant classes?

Study Expected Number of Participants: What prompted the change of studyExpectedNumberOfParticipants from an integer to a string?

@lopierra
Copy link
Member Author

lopierra commented May 10, 2024

@lopierra Participant and Sample IDs: Is it advisable to incorporate both External and Global IDs for participants and samples within the Biospecimen, DataFile, and Participant classes?

Study Expected Number of Participants: What prompted the change of studyExpectedNumberOfParticipants from an integer to a string?

Yes, good point.. we should have both Global and External IDs everywhere.

FOr studyExpectedNumberOfParticipants - I meant to change that back to integer but forgot

I will make these changes now

@lopierra
Copy link
Member Author

@madanucd Sorry, I misunderstood your comment initially.. I do think we should include both Global and External IDs since the Global ID is used by FHIR and the External ID is more recognizable for humans. I added participantGlobalId to Condition for consistency.

Copy link
Contributor

@madanucd madanucd 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 to me.

@lopierra lopierra merged commit d37a446 into main May 13, 2024
2 checks passed
@madanucd madanucd deleted the 2024-04-02 branch July 16, 2024 21:11
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