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

Externally managed content update #8616

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

twhetzel
Copy link
Collaborator

Update the externally managed content. This includes all content, including updates to the rare disease subsets.

Copy link
Collaborator Author

@twhetzel twhetzel left a comment

Choose a reason for hiding this comment

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

I spot checked a few updates against the source data files and it looks good.

@twhetzel twhetzel requested a review from matentzn January 22, 2025 07:13
Copy link
Member

@matentzn matentzn left a comment

Choose a reason for hiding this comment

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

Looks great! Lots of changes from Medgen, and inferred rare (from the epilepsy restructuring), happy here!

subset: gard_rare {source="GARD:2169", source="MONDO:GARD"}
subset: nord_rare {source="MONDO:NORD"}
subset: ordo_disorder {source="Orphanet:1942"}
subset: otar {source="MONDO:OTAR"}
Copy link
Member

Choose a reason for hiding this comment

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

Not related to this PR: it is very important @twhetzel you also have an eye on our commitment to our community to protect the terms they are using, because any time we obsolete a otar, nord, clingen, gard, etc term, we encur bigger costs on the side of these users!

@twhetzel twhetzel merged commit 32c0d53 into master Jan 22, 2025
1 check passed
@twhetzel twhetzel deleted the externally_managed_content-1944fab branch January 22, 2025 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants