Skip to content
This repository has been archived by the owner on May 8, 2024. It is now read-only.

feat: mapping between swerik_ids and biobook references #474

Merged
merged 2 commits into from
Feb 9, 2024
Merged

Conversation

BobBorges
Copy link
Collaborator

Here's a mapping between swerik IDs and the bio-books.

I found 172 individuals in the mp metadata that didn't make it into the known_mp_catalog, thus not in the mapping. 147 of them have a biobook reference that we know of, but the volume / pages would have to be looked up or fetched from wikidata - is that something we want to handle before merging this file? I need to use it for other stuff, but it can sit in PR for a while.

@fredrik1984
Copy link
Collaborator

Ok, good to know! I leave that decision to @MansMeg or @ninpnin

@BobBorges
Copy link
Collaborator Author

@ninpnin Pls take a look /merge

@MansMeg
Copy link
Collaborator

MansMeg commented Feb 9, 2024

Should we open up a new issue to fix the last things?

@ninpnin
Copy link
Collaborator

ninpnin commented Feb 9, 2024

I think we should add a description in corpus/metadata/README.md . Otherwise looks fine by me.

@BobBorges BobBorges merged commit 8ada544 into dev Feb 9, 2024
10 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants