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

obsolete pyogenic autoinflammatory syndrome of childhood #7103

Merged
merged 4 commits into from
Jan 26, 2024

Conversation

nicolevasilevsky
Copy link
Member

close #6789

@nicolevasilevsky nicolevasilevsky marked this pull request as ready for review January 5, 2024 21:44
xref: Orphanet:324942 {source="MONDO:equivalentTo"}
is_a: MONDO:0019751 {source="https://orcid.org/0000-0001-7941-2961", source="https://orcid.org/0000-0002-5002-8648"} ! autoinflammatory syndrome
xref: Orphanet:324942 {source="MONDO:obsoleteEquivalent"}
property_value: IAO:0000231 OMO:0001000 {source="MONDO:excludeHistoricalDisease"}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this the same reason as used in the strategic refinement work? I think is should be: property_value: IAO:0000231 OMO:0001000 {source="MONDO:excludeGrouping"}

Copy link
Member Author

Choose a reason for hiding this comment

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

this was not a grouping class, so it is not appropriate to add that exclusion reason.

Copy link
Member Author

@nicolevasilevsky nicolevasilevsky Jan 12, 2024

Choose a reason for hiding this comment

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

@twhetzel could you please review again and approve and merge?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh, even though the Orphanet page says it is a grouping term: https://www.orpha.net/consor/cgi-bin/OC_Exp.php?lng=EN&Expert=324942?

Copy link
Member Author

Choose a reason for hiding this comment

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

got it! I'll add the exclusion reason

Copy link
Member Author

Choose a reason for hiding this comment

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

ready for review

@sabrinatoro
Copy link
Collaborator

I am questioning whether this term should be obsoleted.

@twhetzel twhetzel marked this pull request as draft January 16, 2024 21:53
@sabrinatoro sabrinatoro added the blocked Something needs to be done before work can proceed label Jan 20, 2024
@sabrinatoro sabrinatoro marked this pull request as ready for review January 25, 2024 23:32
@sabrinatoro
Copy link
Collaborator

@nicolevasilevsky based on the curation call (Jan 25, 2024), I have approved this PR. I also updated from Master and the QCs need to run before we can merge.

Question for you: do you think we should keep the comment? Or should we update it to add something about the term being obsoleted?

@nicolevasilevsky
Copy link
Member Author

Question for you: do you think we should keep the comment? Or should we update it to add something about the term being obsoleted?

Personally, I think if the term is obsoleted, it is not necessary to keep the comment.

@sabrinatoro
Copy link
Collaborator

Question for you: do you think we should keep the comment? Or should we update it to add something about the term being obsoleted?

Personally, I think if the term is obsoleted, it is not necessary to keep the comment.

I agree. Especially this comment makes things confusing. I will remove it. Thank you!

as per discussion on PR
@sabrinatoro sabrinatoro removed the blocked Something needs to be done before work can proceed label Jan 26, 2024
@sabrinatoro sabrinatoro merged commit c78eb90 into master Jan 26, 2024
1 check passed
@nicolevasilevsky nicolevasilevsky deleted the issue-6789 branch January 31, 2024 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Obsolete] MONDO:0957404 pyogenic autoinflammatory syndrome of childhood
3 participants