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 targetPathway to credential, assessment and learning opportunity #555

Closed
stuartasutton opened this issue Nov 25, 2018 · 10 comments

Comments

Projects
None yet
3 participants
@stuartasutton
Copy link
Contributor

commented Nov 25, 2018

Currently, ceterms:targetPathway has a domain of only ceterms:ConditionProfile--a rationale that escapes me since the pathway relationship is simple and seldom likely to be conditioned. As the pathways work has played out, PathwayComponents (prescribed milestones) can represent credentials, assessments, or learning opportunities--as well as other resources. Thus, ceterms:targetPathway should be added to the domains of ceterms:Credential, ceterms:AssessmentProfile, and ceterms:LearningOpportunityProfile so references can be made to pathways in which they are represented as components.

@siuc-nate

This comment has been minimized.

Copy link

commented Dec 3, 2018

I think it's far better from a data integrity standpoint if the pathway references the credential - this will be much more reliable. Otherwise, it will be up to publishers of credentials to update their credentials any time anyone else's pathway is created/modified to reference that credential, leading to incorrect/outdated/misleading information as part of the credential's record itself.

If the data remained solely on the pathway instead, it would be a simple query to get all of the pathways for a credential (simply query for all pathways that reference that credential).

@siuc-nate

This comment has been minimized.

Copy link

commented Dec 3, 2018

@stuartasutton Is the following correct?

Remove:

Subject: ceterms:targetPathway
Predicate: schema:domainIncludes
Object: ceterms:ConditionProfile

Add:

Subject: ceterms:targetPathway
Predicate: schema:domainIncludes
Object: ceterms:Credential (and subclasses)

Add:

Subject: ceterms:targetPathway
Predicate: schema:domainIncludes
Object: ceterms:AssessmentProfile

Add:

Subject: ceterms:targetPathway
Predicate: schema:domainIncludes
Object: ceterms:LearningOpportunityProfile

@stuartasutton

This comment has been minimized.

Copy link
Contributor Author

commented Dec 5, 2018

@siuc-nate, my only reservation at this time is removing the current domainIncludes of ConditionProfile. While I have a hard time imagining an instance of reference to a pathway that is conditioned, that doesn't mean that intuition is correct. I'd prefer adding the others and leaving ConditionProfile for now, and, removing it later when we have some evidence that it's not used.

@siuc-nate

This comment has been minimized.

Copy link

commented Jan 15, 2019

Per our 1/15/2019 meeting:

  • Due to delaying the Pathways issue to the next release, we will hold off on this issue as well.
@stuartasutton

This comment has been minimized.

Copy link
Contributor Author

commented Mar 26, 2019

In working through this addition of targetPathway to credential, assessment, learning opportunity, consider whether the range is actually the Pathway or is it to an aligned PathwayComponent or is it both (targetPathway & targetPathwayComponent). I.e., an instance of AssessmentProfile might have a targetPathwayComponent aligning it to a AssessmentComponent in a Pathway.

@siuc-nate

This comment has been minimized.

Copy link

commented Apr 9, 2019

Per our 4-9-2019 meeting:

Remove:

Subject: ceterms:targetPathway
Predicate: schema:domainIncludes
Object: ceterms:ConditionProfile

Add:

Subject: ceterms:targetPathway
Predicate: schema:domainIncludes
Object: ceterms:Credential (and subclasses)

Add:

Subject: ceterms:targetPathway
Predicate: schema:domainIncludes
Object: ceterms:AssessmentProfile

Add:

Subject: ceterms:targetPathway
Predicate: schema:domainIncludes
Object: ceterms:LearningOpportunityProfile

Remove:

Subject: ceterms:targetPathway
Predicate: rdfs:label
Object: "Career Pathway"

Add:

Subject: ceterms:targetPathway
Predicate: rdfs:label
Object: "Target Pathway"

Remove:

Subject: ceterms:targetPathway
Predicate: rdfs:comment
Object: "Career pathway in which the credential is a potential component."

Add:

Subject: ceterms:targetPathway
Predicate: rdfs:comment
Object: "Pathway in which this resource is a potential component."

@siuc-nate

This comment has been minimized.

Copy link

commented Apr 10, 2019

I have made the changes in pending CTDL. Not noted in history tracking yet.

There are two additional issues:

  • ceterms:targetPathway has no range. Presumably the range should be ceterms:Pathway. Correct?
  • ceterms:targetPathway is still vs:unstable. Do we want to keep it that way for now?

@stuartasutton

@siuc-nate

This comment has been minimized.

Copy link

commented Apr 10, 2019

I see #582 indicates that we should indeed update the range to be ceterms:Pathway.

However, what about the status?

@siuc-nate

This comment has been minimized.

Copy link

commented Apr 10, 2019

Per our 4-10-2019 meeting:
Per Jeanne, this property should indeed be made vs:stable.

@siuc-nate

This comment has been minimized.

Copy link

commented Apr 10, 2019

These changes have been made in pending CTDL and noted in the history tracking.

@siuc-nate siuc-nate closed this Apr 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.