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

MarcXML parser for Exlibris Alma holdings output #255

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

mwarin
Copy link
Contributor

@mwarin mwarin commented Feb 3, 2023

Using this branch to work out how to deal with Exlibris's Alma MarcXML files.

See e.g. https://hathitrust.atlassian.net/browse/DEV-415, https://hathitrust.atlassian.net/browse/DEV-691 , https://hathitrust.atlassian.net/browse/DEV-692, but perhaps most of all DEV-833

@coveralls
Copy link

coveralls commented Feb 3, 2023

Coverage Status

coverage: 95.045%. remained the same
when pulling 182627a on DEV-415-exlibris-sample
into 34e044f on main.

@mwarin mwarin changed the title first stab at an ex-libris holdings xml parser MarcXML parser for Exlibris Alma holdings output Apr 26, 2023
@mwarin
Copy link
Contributor Author

mwarin commented Sep 28, 2023

Failing because this branch needs to rebase or merge in the wait-for/netcat fixes.

Rebased & fixed.

@aelkiss
Copy link
Member

aelkiss commented Jun 28, 2024

I handled some of the item type stuff so we can use the output more directly.

On the file I have only 10 records appear to be govdocs with the given logic (out of 1,931,496 holdings records), so we'll definitely want to check that.

@aelkiss
Copy link
Member

aelkiss commented Jun 28, 2024

Although I don't think we're doing anything with govdoc information right now anyway in holdings, so worst case maybe we can just omit that column.

* to_s gave the field in addition to just the value
* post-zephir processing also accepts "us " and "pr " as US location
  codes
@aelkiss
Copy link
Member

aelkiss commented Jul 1, 2024

I also fixed the govdoc parsing.

@mwarin I figure before merging we should probably have some tests -- anything else you think we should do here?

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.

3 participants