-
Notifications
You must be signed in to change notification settings - Fork 531
add "review" dataset type and related metadata blocks #11753
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
base: develop
Are you sure you want to change the base?
Changes from all commits
bf0f63e
0a48cc8
d287907
c938a7a
1c73793
9138524
e6447b7
99fcede
a7608e9
e6fa18c
016929c
c60359a
b49b522
3de3913
d54ae98
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| ### New Dataset Type: Review | ||
|
|
||
| A new, experimental dataset type called "review" has been added. When this type is published, it will be sent to DataCite as "Other" for resourceTypeGeneral. See #11747. |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,40 @@ | ||
| #metadataBlock name dataverseAlias displayName | ||
| review Review Metadata | ||
| #datasetField name title description watermark fieldType displayOrder displayFormat advancedSearchField allowControlledVocabulary allowmultiples facetable displayoncreate required parent metadatablock_id termURI | ||
| itemReviewed Item Reviewed The item being reviewed none 1 FALSE FALSE FALSE FALSE FALSE FALSE review | ||
| itemReviewedUrl URL The URL of the item being reviewed url 2 FALSE FALSE FALSE FALSE FALSE FALSE itemReviewed review | ||
| itemReviewedType Type The type of the item being reviewed text 3 FALSE TRUE FALSE FALSE FALSE FALSE itemReviewed review | ||
| itemReviewedCitation Citation The full bibliographic citation of the item being reviewed textbox 4 FALSE FALSE FALSE FALSE FALSE FALSE itemReviewed review | ||
| #controlledVocabulary DatasetField Value identifier displayOrder | ||
| itemReviewedType Audiovisual 0 | ||
| itemReviewedType Award 1 | ||
| itemReviewedType Book 2 | ||
| itemReviewedType Book Chapter 3 | ||
| itemReviewedType Collection 4 | ||
| itemReviewedType Computational Notebook 5 | ||
| itemReviewedType Conference Paper 6 | ||
| itemReviewedType Conference Proceeding 7 | ||
| itemReviewedType DataPaper 8 | ||
| itemReviewedType Dataset 9 | ||
| itemReviewedType Dissertation 10 | ||
| itemReviewedType Event 11 | ||
| itemReviewedType Image 12 | ||
| itemReviewedType Interactive Resource 13 | ||
| itemReviewedType Instrument 14 | ||
| itemReviewedType Journal 15 | ||
| itemReviewedType Journal Article 16 | ||
| itemReviewedType Model 17 | ||
| itemReviewedType Output Management Plan 18 | ||
| itemReviewedType Peer Review 19 | ||
| itemReviewedType Physical Object 20 | ||
| itemReviewedType Preprint 21 | ||
| itemReviewedType Project 22 | ||
| itemReviewedType Report 23 | ||
| itemReviewedType Service 24 | ||
| itemReviewedType Software 25 | ||
| itemReviewedType Sound 26 | ||
| itemReviewedType Standard 27 | ||
| itemReviewedType Study Registration 28 | ||
| itemReviewedType Text 29 | ||
| itemReviewedType Workflow 30 | ||
| itemReviewedType Other 31 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -740,8 +740,11 @@ public Map<String, String> getDataCiteMetadata() { | |
|
|
||
| public JsonObject getCSLJsonFormat() { | ||
| CSLItemDataBuilder itemBuilder = new CSLItemDataBuilder(); | ||
| // TODO consider making this a switch | ||
| if (type.equals(DatasetType.DATASET_TYPE_SOFTWARE)) { | ||
| itemBuilder.type(CSLType.SOFTWARE); | ||
| } else if (type.equals(DatasetType.DATASET_TYPE_REVIEW)) { | ||
| itemBuilder.type(CSLType.REVIEW); | ||
|
Comment on lines
+746
to
+747
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @qqmyers I made this change without testing anything because I wasn't sure how to. Can you please advise? I'd like to add something under "how to test" about it.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I guess you can unit test w.r.t. making sure the CSLJson output has the review type if your datasettype is review. Beyond that, the CSL Json is used in front side JavaScript to generate any of thousands of citation formats, but any given format may or may not use CSLType in generating its output. I can't think of any easy way to test that (but that would really be testing the CSL Java and JavaScript libraries - if our code gets the CSLType into the CSL Json output here, things should work).
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ok, thanks. Do you know of any format that uses CSLType? Maybe we can test that type by manually inspecting its output?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't know. AI says APA for the social sciences and Chicago for history and arts, and MLA for humanities are popular for reviews - hopefully that means they check the type.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You can get the CSL format itself via api, e.g. curl
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks, at https://dev1.dataverse.org/api/datasets/:persistentId/versions/1.0/citation/CSL?persistentId=doi%3A10.5072/FK2/RMEZNL I'm getting this: (I need to fix my siteUrl, obviously.) 😅 I guess you're saying ideally "type" would be "review" or at least not "dataset" in that output. It makes me wonder what my change did, if anything. 🤔 Also, thanks for the heads up about the typo in the guides.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I meant to say you've given me an API endpoint to dig into. Thanks! |
||
| } else { | ||
| itemBuilder.type(CSLType.DATASET); | ||
| } | ||
|
|
||

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.
Shouldn't this say ("Dataset", "Software", "Workflow", "Other")? Review is sent to Datacite as "Other"
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.
Hmm, good point. Maybe I'll change this to "certain types" and link to the table I made elsewhere, or something. Thanks! 😅