Skip to content

Create DOSDP entity_response_quality_in_location#399

Merged
aleixpuigb merged 6 commits into
masterfrom
391_DOSDP_entity_response_quality_in_location
Oct 7, 2025
Merged

Create DOSDP entity_response_quality_in_location#399
aleixpuigb merged 6 commits into
masterfrom
391_DOSDP_entity_response_quality_in_location

Conversation

@aleixpuigb

Copy link
Copy Markdown
Collaborator

Create DOSDP entity_response_quality_in_location and use to add new term OBA:2090002 'level of anti-Helicobacter pylori IgG in blood'.
Fixes #391

Introduces a new DOSDP pattern for traits describing the quality of an entity in response to a stimulus and location. Adds corresponding TSV data for anti-Helicobacter pylori IgG in blood.
Changed NCIT import references to use the new GitHub raw URL
@aleixpuigb aleixpuigb self-assigned this Oct 1, 2025
@aleixpuigb aleixpuigb marked this pull request as draft October 1, 2025 15:59
Replaced the use of RO_0002503 ('in_response_to') with RO_0020329 ('indicates_response_to') in ontology definitions and DOSDP pattern.
value: xrefs

equivalentTo:
text: "%s and 'characteristic_of' some (%s and ('indicates_response_to' some %s) and ('part_of' some %s))"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I am not sure that indicates_response_to is the best object property I can use, but I could not find a better one. Ideally I would like in_response _to. @ar-ibrahim @matentzn any ideas?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

indicates_response_to is defined by RO as "A relationship between a biomarker 'b' and an exposure agent 'e', where 'b' is used to show that a biological response to 'e' has occurred." and comments that 'the exposure agent' can be an organism. And this fits with the terms we want to create with this new pattern: 'biomarker' IgG released as a biological response to 'exposure agent' Helicobacter pylori, a bacterial organism. We can also ask for suggestions in RO slack but I think indicates_response_to is an appropriate relation to use in this context.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks Arwa!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In order to not block the universe from doing good work, I stay out of RO discussions. Looks good enough.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looking at the pattern I think would want to use it for something like 'liver Cyp1a1 level after dioxin exposure' and based on the RO term label I think that would be okay. The one thing that gives me pause is that indicates_response_to sits under indicates_clinical_effect_of but I think this is more of an issue with RO. I would consider indicates_response_to to be broader than 'clinical effect'. But I also don't see anything in the clinical effect parent that specifies what 'clinical' means so possibly its just a bad label in RO.
TL/DR - I don't see anything better.

@aleixpuigb aleixpuigb marked this pull request as ready for review October 7, 2025 10:18

@matentzn matentzn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I am fine with this - but I am not a great person to review such things. Ray was the best, now he is of the project, I think you could do a friendly reach out to Sue Bello on slack.

Comment thread src/patterns/dosdp-patterns/entity_response_quality_in_location.yaml Outdated
value: xrefs

equivalentTo:
text: "%s and 'characteristic_of' some (%s and ('indicates_response_to' some %s) and ('part_of' some %s))"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In order to not block the universe from doing good work, I stay out of RO discussions. Looks good enough.

@aleixpuigb aleixpuigb requested a review from sbello October 7, 2025 14:35

@sbello sbello left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think this looks good. I don't see a better RO term to use

value: xrefs

equivalentTo:
text: "%s and 'characteristic_of' some (%s and ('indicates_response_to' some %s) and ('part_of' some %s))"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looking at the pattern I think would want to use it for something like 'liver Cyp1a1 level after dioxin exposure' and based on the RO term label I think that would be okay. The one thing that gives me pause is that indicates_response_to sits under indicates_clinical_effect_of but I think this is more of an issue with RO. I would consider indicates_response_to to be broader than 'clinical effect'. But I also don't see anything in the clinical effect parent that specifies what 'clinical' means so possibly its just a bad label in RO.
TL/DR - I don't see anything better.

@ar-ibrahim ar-ibrahim left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

No further comments to add. All looks good Aleix.

@aleixpuigb

Copy link
Copy Markdown
Collaborator Author

I think this looks good. I don't see a better RO term to use

Thank you for the feedback!

@aleixpuigb aleixpuigb merged commit a5b0be8 into master Oct 7, 2025
@aleixpuigb aleixpuigb deleted the 391_DOSDP_entity_response_quality_in_location branch October 7, 2025 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[NTR] amount in blood terms

4 participants