Skip to content
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

Update Project MUSE ONIX subject output logic #299

Merged
merged 7 commits into from
Oct 20, 2021

Conversation

rhigman
Copy link
Member

@rhigman rhigman commented Oct 18, 2021

In a meeting between members of Thoth and Project MUSE on 2021-09-29 (minutes), MUSE mentioned that they require at least one BIC or BISAC subject code in each ONIX record, and cannot process records containing "keyword" subject codes. They reported no problems with the minor changes requested by JSTOR, suggesting that these are acceptable to merge back in to the MUSE ONIX output.

This PR makes these changes, and marks that MUSE output is also acceptable to JSTOR, and OAPEN output is also acceptable to DOAB.

@rhigman rhigman requested a review from ja573 October 18, 2021 11:56
@rhigman rhigman linked an issue Oct 19, 2021 that may be closed by this pull request
write_element_block("SubjectCode", w, |w| {
w.write(XmlEvent::Characters(&subject.subject_code))
// Project MUSE can't process records containing keywords
if subject.subject_type != SubjectType::KEYWORD {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also ignore SubjectType::CUSTOM?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Project MUSE explictly confirmed that "outputting "custom" subject codes with SubjectSchemeIdentifier B2 is acceptable and will not cause us any problems".

The broader question (which I think came up in passing at our recent meeting) is whether we should be outputting SubjectType::CUSTOM for any of the flavours, since they're (for OBP at least) internal-only and therefore irrelevant to distributors (and possibly not ingested by them). At the moment, we output it for all flavours except OAPEN.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In that case let's leave them. I guess there's no harm in outputting them if they're going to be ignored

ja573
ja573 previously approved these changes Oct 20, 2021
@rhigman rhigman merged commit cf91135 into develop Oct 20, 2021
@rhigman rhigman deleted the feature/102_muse_subjects branch October 20, 2021 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Output to DOAB
2 participants