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

IDMP-641 add CDC - PHIN Race and Ethnicity with agencies #451

Merged
merged 25 commits into from
Sep 20, 2023
Merged

Conversation

tw-osthus
Copy link
Contributor

Description:

  • added CDC, PHIN-VADS to na registries
  • added Race and Ethnicity vocabulary

Copy link
Contributor

@mereolog mereolog left a comment

Choose a reason for hiding this comment

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

It is not possible to check consistency of https://spec.pistoiaalliance.org/idmp/ontology/ISO/20230814/NorthAmericanJurisdiction/PublicHealthInformationNetwork/ - see below the error thrown by Pellet:

ERROR  12:29:52  An error occurred during reasoning: Unsupported axiom: Ignoring transitivity and/or complex subproperty axioms for hasBroaderEntry.
org.mindswap.pellet.exceptions.UnsupportedFeatureException: Unsupported axiom: Ignoring transitivity and/or complex subproperty axioms for hasBroaderEntry
	at org.mindswap.pellet.RBox.ignoreTransitivity(RBox.java:813) ~[na:na]
	at org.mindswap.pellet.RBox.prepare(RBox.java:612) ~[na:na]
	at org.mindswap.pellet.KnowledgeBase.prepare(KnowledgeBase.java:1776) ~[na:na]
	at org.mindswap.pellet.KnowledgeBase.consistency(KnowledgeBase.java:1969) ~[na:na]
	at org.mindswap.pellet.KnowledgeBase.isConsistent(KnowledgeBase.java:2061) ~[na:na]
	at org.mindswap.pellet.KnowledgeBase.ensureConsistency(KnowledgeBase.java:2075) ~[na:na]
	at org.mindswap.pellet.KnowledgeBase.classify(KnowledgeBase.java:2083) ~[na:na]
	at com.clarkparsia.pellet.owlapiv3.PelletReasoner.precomputeInferences(PelletReasoner.java:1067) ~[na:na]
	at org.protege.editor.owl.model.inference.OWLReasonerManagerImpl$ClassificationRunner.precompute(OWLReasonerManagerImpl.java:447) ~[na:na]
	at org.protege.editor.owl.model.inference.OWLReasonerManagerImpl$ClassificationRunner.run(OWLReasonerManagerImpl.java:388) ~[na:na]
	at java.base/java.lang.Thread.run(Unknown Source) ~[na:na]
   WARN  12:29:52  Protege terminated reasoner.

@mereolog
Copy link
Contributor

See also #454 (comment) - a similar comment applies here.

…hasNarrowerEntry, it does not work with transitive properties

Signed-off-by: Thomas Weber <[email protected]>
@tw-osthus
Copy link
Contributor Author

needs fix of master branch after merge

Copy link
Contributor

@mereolog mereolog left a comment

Choose a reason for hiding this comment

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

There is a couple of typos reported in the other comments.


<owl:NamedIndividual rdf:about="&idmp-phin;VADSCodeSystem-RaceAndEthnicityCDC">
<rdfs:label>PHIN-VADS code system - Race and Ethnicity - CDC</rdfs:label>
<rdfs:type rdf:resource="&idmp-phin;VADSCodeSystem"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

This line should read: <rdf:type rdf:resource="&idmp-phin;VADSCodeSystem"/>

<owl:Class rdf:about="&idmp-phin;VADSName">
<rdfs:subClassOf rdf:resource="&cmns-dsg;Name"/>
<rdfs:label>PHIN-VADS name</rdfs:label>
<rdf:subClassOf>
Copy link
Contributor

Choose a reason for hiding this comment

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

This line should read <rdfs:subClassOf>.

<owl:allValuesFrom rdf:resource="&idmp-phin;VADSCodeSystem"/>
</owl:Restriction>
</rdf:subClassOf>
<rdf:subClassOf>
Copy link
Contributor

Choose a reason for hiding this comment

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

This line should read <rdfs:subClassOf>.

<owl:onProperty rdf:resource="&cmns-col;isIncludedIn"/>
<owl:allValuesFrom rdf:resource="&idmp-phin;VADSCodeSystem"/>
</owl:Restriction>
</rdf:subClassOf>
Copy link
Contributor

Choose a reason for hiding this comment

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

This line should read </rdfs:subClassOf>.

<owl:onProperty rdf:resource="&cmns-dsg;denotes"/>
<owl:allValuesFrom rdf:resource="&idmp-phin;VADSCodeSystemConcept"/>
</owl:Restriction>
</rdf:subClassOf>
Copy link
Contributor

Choose a reason for hiding this comment

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

This line should read </rdfs:subClassOf>.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@mereolog
Copy link
Contributor

ERROR 11:19:45 An error occurred during reasoning: Cannot do reasoning with inconsistent ontologies!

Reason for inconsistency: Literal value "https://phinvads.cdc.gov/vads/ViewCodeSystem.action?id=2.16.840.1.113883.6.238"^^string does not belong to datatype anyURI.
org.mindswap.pellet.exceptions.InconsistentOntologyException: Cannot do reasoning with inconsistent ontologies!
Reason for inconsistency: Literal value "https://phinvads.cdc.gov/vads/ViewCodeSystem.action?id=2.16.840.1.113883.6.238"^^string does not belong to datatype anyURI

Comment on lines +111 to +112
<owl:imports rdf:resource="https://spec.pistoiaalliance.org/idmp/ontology/ISO/NorthAmericanJurisdiction/CommonTerminologyCriteriaForAdverseEvents/"/>
<owl:imports rdf:resource="https://spec.pistoiaalliance.org/idmp/ontology/ISO/NorthAmericanJurisdiction/PublicHealthInformationNetwork/"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

These two ontologies should be "registered" in AboutIDMPDev-ReferenceIndividuals.rdf as well because all and only ontologies "registered" there are subject to hygiene tests.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

tw-osthus and others added 4 commits September 18, 2023 10:56
Signed-off-by: Thomas Weber <[email protected]>
…value domain from ISO 21090

Signed-off-by: Elisa Kendall <[email protected]>
…D, further defined the code for the code set, replaced min 1 cardinality restrictions with some values from for improved performance, completed other concepts and corrected spelling

Signed-off-by: Elisa Kendall <[email protected]>
ElisaKendall
ElisaKendall previously approved these changes Sep 20, 2023
Signed-off-by: Elisa Kendall <[email protected]>
Copy link
Contributor

@ElisaKendall ElisaKendall left a comment

Choose a reason for hiding this comment

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

Approved with the changes made.

@mereolog mereolog merged commit b490221 into master Sep 20, 2023
6 checks passed
@mereolog mereolog deleted the IDMP-641 branch September 20, 2023 14:59
@mereolog mereolog added this to the Release 0.5.0 milestone Sep 22, 2023
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