-
Notifications
You must be signed in to change notification settings - Fork 38
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 Codemeta terms #276
Add Codemeta terms #276
Conversation
Codemeta Namespace boldly assumed to be w3id-based rather than github.io see codemeta/codemeta#216 Added Biochemas namespace comment, see BioSchemas/specifications#653
See issue #275
Not sure if we should add this for 1.2 when we don't have a corresponding section explaining when to use |
@dgarijo to double-check the PIDs |
I added a few comments @stain. In Codemeta we resolved the resolution and content negotiation of w3id.org/codemeta, but at this time, the id is still I suggest using for now the current Codemeta id, even if it's not machine readable |
It's OK, better remain compatible for now and use codemeta.github.io namespace. I can add the machine readable definitions to our core Profile Crate for 1.2-DRAFT, but we would need to add also human-readable text to explain also why these URLs are 404 Not Found. |
As shown in codemeta/codemeta#360 my proposal is to move it to https://w3id.org/codemeta/terms/ It would be great if you drop a +1 to show your support to this change |
Hopefully by June 20th we'll get it solved. But community standards are slow sometimes. |
I've added machine readable version in https://github.com/ResearchObject/ro-crate/blob/codemeta/docs/1.2-DRAFT/ro-crate-metadata.json#L736 , which you are free to adapt/correct. Unsure about if Edit: Most of them say |
I will review when time allows. Thanks! |
Ideally from https://www.researchobject.org/ro-crate/specification/1.2-DRAFT/appendix/jsonld.html#extending-ro-crate we should have a However, while each term is defined on https://codemeta.github.io/terms/ textually, there are no HTML anchors there, so https://codemeta.github.io/terms/#developmentStatus for instance would not help much and would still have that dangerous And we can't have multiple @dgarijo Could we use the {
"@id": "https://codemeta.github.io/terms/developmentStatus",
"@type": ["DefinedTerm", "rdf:Property"],
"name": "developmentStatus",
"termCode": "developmentStatus",
"description": "Description of development status, e.g. Active, inactive, suspended. See repostatus.org",
"domainIncludes": {"@id": "http://schema.org/SoftwareSourceCode"},
"rangeIncludes": {"@id": "http://schema.org/Text"}
"sameAs": "https://w3id.org/codemeta#developmentStatus"
}
While https://w3id.org/codemeta#developmentStatus does resolve to a HTML page you still have to scroll down and look for "developmentStatus" but at least it's a persistent URL for each term, and also in the promised namespace-to-come. Related question is if "sameAs": { "@id": "https://w3id.org/codemeta#developmentStatus" } |
@dgarijo could you also check if I got the |
I got it wrong above, I see codemeta/codemeta#360 (reply in thread) suggests https://w3id.org/codemeta/terms/ as namespace, so I've added that, e.g. {
"@id": "https://codemeta.github.io/terms/buildInstructions",
"@type": ["DefinedTerm", "rdf:Property"],
"sameAs": "https://w3id.org/codemeta/terms/buildInstructions"
} |
@dgarijo Can you confirm I got this right now..? |
Will complete a review by the end of the week. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating this. It looks good.
I think some of the domains may be also SoftwareApplication, but let us leave it consistent with https://github.com/codemeta/codemeta/blob/master/crosswalk.csv#L63.
Please note that developmentStatus is under discussion, the range may change to a URL. But I don't know when.
Nice idea adding the sameAs to the /terms
ns. I will try and resurrect the thread once September starts.
See namespace considerations in #275 before merging.