CredentialEngine / Schema-Development

Development of the vocabularies for the CTI models
14 stars 8 forks source link

Changes to integrate Pathway constraints model into ConditionProfile and related resources #848

Open stuartasutton opened 2 years ago

stuartasutton commented 2 years ago

In order not to have to return to ceterms:prerequisite immediately after these discussions are concluded, we need to also consider the addition of ceterms:ConditionProfile to its range. This proposal stems from earlier discussions (at Phil's suggestion) that aspects of the model for ceterms:PathwayComponents conditions and constraints are also applicable to ceterms:Course and ceterms:LearningOpportunity through ceterms:ConditionProfile:

  1. ceterms:hasConstraint
  2. ceterms:logicalOperator
  3. ceterms:Constraint

Also add ceterms:targetCourse to ceterms:ConditionProfile as a subproperty of ceterms:targetLearningOpportunity.

The Hutchison Welding Technology prerequisites for course EN101: English Composition II illustrate the need:

Prerequisites: EN101 English Composition IA with a grade of C or higher, or EN101H English Comp IA with a grade of C or higher, or EN100 English Comp IB with a grade of C or higher.

Slight tweaks to the definitions of affected properties are likely necessary.

siuc-nate commented 2 years ago

I would suggest treating prerequisite the same way we treat requires and entryCondition in terms of domain and range, as that would avoid future issues there (though it does feel even more redundant with entryCondition that way, it will at least then behave consistently with the other properties).

Adding those two pathway-related properties to Condition Profile makes sense, at least on the surface, but:

Also add ceterms:targetCourse to ceterms:ConditionProfile as a subproperty of ceterms:targetLearningOpportunity.

There is no targetCourse property, and targetLearningOpportunity already explicitly covers Course, so I don't see a need for this.

siuc-nate commented 2 months ago

Discussion of this has picked up (somewhat) in #947