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

add 'human traveler role'. Fixes #191 #261

Merged
merged 2 commits into from
Jan 17, 2025
Merged

add 'human traveler role'. Fixes #191 #261

merged 2 commits into from
Jan 17, 2025

Conversation

hoganwr
Copy link
Contributor

@hoganwr hoganwr commented Jan 17, 2025

No description provided.

@hoganwr hoganwr requested review from dexink and dillerm January 17, 2025 17:56
Copy link
Collaborator

Choose a reason for hiding this comment

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

duplicate imported term from apollo

@@ -3588,4 +3600,4 @@ AnnotationAssertion(rdfs:label obo:IAO_0000412 "imported from"@en)
AnnotationAssertion(owl:deprecated obo:OMIABIS_0000060 "true"^^xsd:boolean)
AnnotationAssertion(owl:deprecated obo:OMIABIS_0001025 "true"^^xsd:boolean)
AnnotationAssertion(owl:deprecated obo:OMIABIS_0001035 "true"^^xsd:boolean)
)
)
Copy link
Collaborator

@dexink dexink Jan 17, 2025

Choose a reason for hiding this comment

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

unintended deletion and re-addition of end parenthesis. appears to have occurred automatically, but needs investigated.
while the intention of this commit was only the addition of the "human traveler role" class, the addition of an obsoleted class (replacing the equivalent non-obsoleted class) and the replacement of a separate obsolete class were also completed.

Copy link
Collaborator

@dillerm dillerm Jan 17, 2025

Choose a reason for hiding this comment

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

I think this was intended because the document had lines added to it, therefore requiring that the closing parenthesis be moved. Also, an obsolete class was not added; it's label was corrected to include 'obsolete'.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The other lines involving the OP are comments. Lines beginning with '#' are comments. I think this looks good. I fixed the issue with the redundant import in external_terms.tsv.

Anything else, or all good now?

Copy link
Collaborator

@dexink dexink left a comment

Choose a reason for hiding this comment

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

see individual comments

Human travel was already imported. Although the ODK import process is not impacted by the redundancy, it's not great having it here.
@@ -500,7 +501,7 @@ SubObjectPropertyOf(obo:OMIABIS_0000008 oboInOwl:ObsoleteProperty)
AnnotationAssertion(owl:deprecated obo:OMIABIS_0000009 "true"^^xsd:boolean)
SubObjectPropertyOf(obo:OMIABIS_0000009 oboInOwl:ObsoleteProperty)

# Object Property: obo:OMRSE_00000020 (is-aggregate-of)
# Object Property: obo:OMRSE_00000020 (obsolete is-aggregate-of)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is just a change in a comment. Lines beginning with '#' are comments. The OWL API has updated the "header" comment for classes and OPs to include the labels. So as we change the labels, it will periodically rewrite these comment lines. Inconsequential change.

@@ -823,7 +824,7 @@ AnnotationAssertion(rdfs:label obo:OMRSE_00000019 "integrated delivery network"@
SubClassOf(obo:OMRSE_00000019 obo:OMRSE_00000015)
SubClassOf(obo:OMRSE_00000019 ObjectSomeValuesFrom(obo:RO_0000052 ObjectIntersectionOf(obo:OBI_0000245 ObjectAllValuesFrom(obo:OBI_0001688 obo:OBI_0000245))))

# Class: obo:OMRSE_00000022 (collection of organisms)
# Class: obo:OMRSE_00000022 (aggregate of objectual organisms)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is just a change in a comment. Lines beginning with '#' are comments. The OWL API has updated the "header" comment for classes and OPs to include the labels. So as we change the labels, it will periodically rewrite these comment lines. Inconsequential change.

@hoganwr hoganwr merged commit 576778e into main Jan 17, 2025
1 check passed
@hoganwr hoganwr deleted the humam-traveler-role branch January 17, 2025 18:58
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