-
Notifications
You must be signed in to change notification settings - Fork 0
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
CDC 2.5 next profile: publication and citation information #66
Comments
Original comment by Taina Jääskeläinen. CDC Upgrade project thought the publication information could be documented in DDI like this. Arrgghh, cannot find a way to copy the table where the x-paths are. @darrenbell2 please take a look at the project report. The x-paths are on page 102 in the table, for both DDI 2.5 and DDI-L. https://docs.google.com/document/d/1oSveHkGZJjDPOH2muMhsjWQz7kb00SB6/edit |
Original comment by Darren Bell (GitHub: darrenbell2). @TainaFSD I have consulted p.102 at ttps://docs.google.com/document/d/1oSveHkGZJjDPOH2muMhsjWQz7kb00SB6/edit but I am not sure which XPaths you need to be included. |
Original comment by Taina Jääskeläinen. All of them. I put in some Usage and Required info below as well. codeBook/stdyDscr/othrStdyMat/relPubl
codeBook/stdyDscr/othrStdyMat/relPubl/citation/titlStmt/title
codeBook/stdyDscr/othrStdyMat/relPubl/citation/titlStmt/IDNo
codeBook/stdyDscr/othrStdyMat/relPubl/citation/titlStmt/IDNo/@agency
|
Original comment by Darren Bell (GitHub: darrenbell2). @TainaFSD The changes Draft committed at https://github.com/cessda/cessda.metadata.profiles/blob/mdo-d6/ I cannot find “r:OtherMaterialScheme” in schema, so I am using r:OtherMaterial Added the following to cdc25_profile.xml; cdc25_profile_mono.xml; cdc_122_profile.xml; cdc_122_profile_mono.xml.
Added the following to cdc32_profile.xml.
|
Original comment by Taina Jääskeläinen. cdc_122_profile.xml
cdc_122_profile_mono.xml
cdc25_profile
cdc25_profile_mono
|
Original comment by Darren Bell (GitHub: darrenbell2).
I’ll commit at end of the day for final approval. Thanks, Darren |
Original comment by Darren Bell (GitHub: darrenbell2). Fixed issue 66 in 5 x CDC profiles |
Original comment by Darren Bell (GitHub: darrenbell2). Hi @TainaFSD I’ve committed the final amendments in https://github.com/cessda/cessda.metadata.profiles/blob/mdo-d6/ |
Original comment by Taina Jääskeläinen. Yes, I will close. Language attributes now seem to be added. The issue whether relPub should be there or not is too technical for me. @KatjaMoilanen Please add info here if you disagree with closing the issue. |
Original comment by Katja Moilanen. If the relPubl does not hold the bibliographic citation of the related publication, then the only information about the related publication will be title or/and PID. As far as I know, the only information that the most of the SPs have about the related publications is bibliographic citation as a string (they do not even have a title as a separate string). Using relPubl for bibliographic citation makes it easier for SPs to provide this information (using citation/biblCit would require giving title). This means that the relPubl should be then specified as a content element. |
Original comment by Darren Bell (GitHub: darrenbell2). Hi @KatjaMoilanen (and @TainaFSD @MortenSikt @Hilde Orten ).
Are you saying that we should have this instead?
This is fine but in this case, where should the PID go? I guess we could use
Thanks, Darren |
Original comment by Katja Moilanen. As Taina originally suggested, all of them are needed: /codeBook/stdyDscr/othrStdyMat/relPubl (recommended) /codeBook/stdyDscr/othrStdyMat/relPubl/@xml:lang (required if 'relPubl' is present) /codeBook/stdyDscr/othrStdyMat/relPubl/citation/titlStmt/titl (recommended) /codeBook/stdyDscr/othrStdyMat/relPubl/citation/titlStmt/titl/@xml:lang (required if 'titl' is present) /codeBook/stdyDscr/othrStdyMat/relPubl/citation/titlStmt/IDNo (optional) /codeBook/stdyDscr/othrStdyMat/relPubl/citation/titlStmt/IDNo/@agency (required if 'IDNo' is present) |
Original comment by Darren Bell (GitHub: darrenbell2). OK thanks @KatjaMoilanen I’ll make the changes and commit at end of today. Thanks, Darren |
Original comment by Darren Bell (GitHub: darrenbell2). In summary, this is what I have now implemented for the CDC profiles:
|
Original comment by Darren Bell (GitHub: darrenbell2). Not sure why this was in state 'Closed' |
Original comment by Darren Bell (GitHub: darrenbell2). fixed issue #66 |
Original comment by Darren Bell (GitHub: darrenbell2). @KatjaMoilanen @TainaFSD can you confirm this is OK to close. Changes can be reviewed at https://github.com/cessda/cessda.metadata.profiles/blob/mdo-d6/ |
Original comment by Taina Jääskeläinen. cdc32_profile.xml ddi:DDIInstance/s:StudyUnit/r:OtherMaterial/r:Citation/r:InternationalIdentifier/r:IdentifierContent
di:DDIInstance/s:StudyUnit/r:OtherMaterial/r:Citation/r:InternationalIdentifier/r:ManagingAgency
ddi:DDIInstance/s:StudyUnit/r:OtherMaterial/r:Citation/r:Title/r:String
Just some tiny issues: cdc_122_profile.xml and cdc25_profile.xml /codeBook/stdyDscr/othrStdyMat/relPubl/@xml:lang Usage information
/codeBook/stdyDscr/othrStdyMat/relPubl/citation/titlStmt/titl
|
Original comment by Katja Moilanen. In addition to issues reported by Taina, I have one small suggestion:
Usage information: Instead of the sentence “Simple description/title of the publication related to the data, if the more detailed relPubl/citation/titlStmt/titl (and its child elements) are not being used.” I suggest “(Bibliographic) citation of the publication related to the data, if the more detailed relPubl/citation/ with its child elements are not being used.” |
Original comment by Taina Jääskeläinen. So the change would be:
|
Original comment by Taina Jääskeläinen. @KatjaMoilanen What would be the corresponding elements for the distDate and holdings URI in DDI 3.2 in CMM? |
Original comment by Katja Moilanen. DDI2.5 DDI2.6 DDI3.2 DDI3.3 DDI3.2 needs also DDI3.3 needs also |
Original comment by Darren Bell (GitHub: darrenbell2). cdc25_profile.xml, cdc25_profile_mono.xml, cdc122_profile.xml, cdc122_profile_mono.xml: Removed /codeBook/stdyDscr/othrStdyMat/relPubl/citation/titlStmt/titl and ..@xml:lang NOW READS: |
Original comment by Darren Bell (GitHub: darrenbell2). cdc32_profile.xml ddi:DDIInstance/g:ResourcePackage/r:OtherMaterial/r:TypeOfMaterial to be set to “Related Publication” CMM 9.2 RECOMMENDED |
Original comment by Darren Bell (GitHub: darrenbell2). Committed changes to cdc25_profile.xml, cdc25_profile_mono.xml, cdc122_profile.xml, cdc122_profile_mono.xml, cdc32_profile.xml Will reflect these in changes in 2.6 and 3.3 profiles as well |
Original comment by Darren Bell (GitHub: darrenbell2). Fixed issue #66 |
Original report on BitBucket by Taina Jääskeläinen.
OpenAire would require publication PID.
Elements relevant for publications and citation to be added to the profile. Documentation to recommend adding publication PID, if known.
So far, I’m not adding issues for 1.2.2 profiles since it needs to be checked first whether there will be any SPs still using those in 2022.
Dataset is exposed in OpenAIRE if at least one these two prerequisite are fulfilled:
Source: https://guidelines.openaire.eu/en/latest/data/use_of_oai_pmh.html#d-setcontent
CDC User Group decision: In any case, regardless of OpenAire, CDC would like to display citations for publications whenever available. Citation is enough for this purpose.
Citations and other publication information should be ‘Optional’ since not all data have related publications.
I will copy the OpenAire requirements in another comment.
The text was updated successfully, but these errors were encountered: