-
Notifications
You must be signed in to change notification settings - Fork 5
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
json ld validation errors #197
Comments
@colugo - one more for you. |
@onthebreeze Within untp-core, the following class declaration is repeated across a number if "identifiers":
Thus the declaration is that a untp-core:Standard is a subclass of untp-core:StandardID, and that doesn't make sense to me. While an instance of a untp-core:StandardID may classified independently of a untp-core:Standard instance, why would an identifier for a standard be the parent of the standard within a classification system? In the real world a "standard" encapsulates multiple concepts but it isn't in itself encapsulated by an "Identifier" concept, albeit it may have a "property" or "attribute" that is an identifier. Regarding "Standard" and "Identifier" I don't believe parent/child inheritance applies but instead and identifier is an object property (link) or a datatype property (attribute) of a standard (in this one example from untp-core). In case I'm missing something here, what is the rationale for making untp-core:StandardID a parent class for untp-core:Standard? |
@onthebreeze While we are onto identifiers, untp-core:Identifier seems like it should be a parent class for untp:FacilityID. Currently untp-core:FacilityID is defined in the json ld as:
Whereas it more logically would be described as:
|
Yes. Changed it so that "product", "facility" etc have parent "entity" |
@onthebreeze I noted some validation errors in the untp-core JSON LD:
When I validate the JSON LD at: schema.org json validator
I get some errors for: https://jargon.sh/user/unece/untp-core/v/working/artefacts/jsonld/render.jsonld.
It appears the issue is a lack of @type in relation to schema:rangeIncludes and schema:domainIncludes.
The text was updated successfully, but these errors were encountered: