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

Change CONFIDENCE_ASSESSMENT from a THING to an ENTITY. #783

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

kquick
Copy link
Contributor

@kquick kquick commented Oct 5, 2022

A confidence assessment is something which exists, and is therefore reasonable to be an entity.

In addition, it has a wasGeneratedBy property that is not defined for THING but which is defined for ENTITY. This issue was detected by ASSIST-DV with the enhancements in PR#782 with the following report:

ERROR: Property http://arcos.rack/PROV-S#wasGeneratedBy was referenced on class http://arcos.rack/CONFIDENCE#CONFIDENCE_ASSESSMENT, but that property is only defined for the unrelated class http://arcos.rack/PROV-S#ENTITY

A confidence assessment is something which exists, and is therefore reasonable to
be an entity.

In addition, it has a wasGeneratedBy property that is not defined for THING but
which is defined for ENTITY.  This issue was detected by ASSIST-DV with the
enhancements in PR#782 with the following report:

> ERROR: Property http://arcos.rack/PROV-S#wasGeneratedBy was referenced on class http://arcos.rack/CONFIDENCE#CONFIDENCE_ASSESSMENT, but that property is only defined for the unrelated class http://arcos.rack/PROV-S#ENTITY
@kquick kquick requested a review from kityansiu October 5, 2022 15:39
@kquick kquick merged commit b85a66b into master Oct 5, 2022
@Ptival Ptival deleted the conf_assess_entity branch October 13, 2022 17:51
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.

None yet

2 participants