-
Notifications
You must be signed in to change notification settings - Fork 93
Update crosswalk for Citation File Format version 1.2.0 #263
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
Conversation
|
In the process of creating a new crosswalk (citation-file-format/citation-file-format#265) for the new version 1.2.0 of the Citation File Format, some questions have popped up which I'd like to have your input on, so that the crosswalk is both correct and as usable as possible:
Sorry for this long list of Qs, and thanks in advance for taking the time to answer them! Pinging @tmorrell, as you have recently worked on the CM-CFF converter (action), and @mfenner as liaison between FORCE11 SCIWG, CM and CFF (and everything else) 🙂. |
|
I don't have all the answers, but I have some thoughts about the crosswalk. I think the intention for the crosswalk is both humans and machines. It's useful to have something that is easily readable, and I've used the crosswalk as the basis for https://github.com/caltechlibrary/convert_codemeta. I had to add additional code to handle the edge cases where logic was needed. My thought is there is a limit to how much we can pack into a csv file, and we shouldn't try to overload it too much. I don't think it's possible to have this crosswalk be completely machine actionable in both directions. For different levels of properties, I've been using . notation (e.g. person.given-names). It seems more readable to me, although I know there are couple of variants in the crosswalk table. At the CodeMeta level we should probably pick between |
|
I don't have all the answers either, but machine-readable crosswalks are hard. Formally, I think the preferred way to tackle such a 'schema integration' problem would be to have both frameworks rigorously defined as RDF (e.g. JSON-LD), and express the mapping in OWL ontology, which permits notions of JSON-LD's So in general I view the crosswalk as a primarily human-facing document for the time being. |
|
Thanks @tmorrell and @cboettig, this helps a lot! I'll stick to the I've also opened #265 as one way of deferring the solution to these issues to addtl. documentation. |
- Possible solution for #265
|
I think the current state is good to go, but before I mark this "Ready for review", can you give this a pass please, @jspaaks, to make sure that there aren't any blatant errors? Thanks! |
|
Thanks for your reviews @moranegg and @jezcope! 🙏 I've made the necessary changes (but left the comments open for you to resolve). |
moranegg
left a comment
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.
Great work on this crosswalk!
We discovered in today's call (FORCE11 hackathon) that citation in schema.org
https://schema.org/citation
"A citation or reference to another creative work, such as another publication, web page, scholarly article, etc."
The emphasis here is another work, so wouldn't be the preferred way to cite this work.
We should use for preferred citation this pending property:
https://schema.org/usageInfo
Or create a new property in CodeMeta for preferred citation which is not expressed today.
|
@moranegg See the related discussion in SOSO for schema.org citations: ESIPFed/science-on-schema.org#42 |
Thanks for this comment, @moranegg! Interesting.
In order to not further hinder a merge of this PR, I'll remove the |
|
Hi @moranegg, thanks for your review. I've now removed the I've also removed the I hope this can be merged now :). |
|
This PR can be merged directly into V2.1 (crosswalk release) |
This PR replaces the out-of-date crosswalk for CFF 1.0.2 with one for the new version 1.2.0.