Skip to content

Commit

Permalink
Add optional ORCID provenance column (#31)
Browse files Browse the repository at this point in the history
Closes #30 

This PR does the following:

- [x] Add the ability to add an **optional** ORCID column (labeled
"contributor") to each curated TSV
- [x] Adds two example ORCID annotations
- [x] Updates the output to incorporate ORCID provenance in SSSOM and
ontology output
- [x] Update data integrity tests (will do this if the PR gets positive
feedback and the questions below are answered)
 
It has the minor drawback of creating a big diff, but it's mostly
localized to generated artifacts.

Questions before merging this, if everyone thinks it's a good idea:

1. Should we enforce that the `contributor` column is always present,
even if it's mostly empty?
2. Should we curate a list of ORCID->Name mappings to make these outputs
(namely, the ontology) nicer to read in Protege? It's also possible to
quickly interact with the ORCID REST API to look up names as an
alternative.
  • Loading branch information
cthoyt authored Jan 8, 2025
1 parent 8fbee9a commit 20dc268
Show file tree
Hide file tree
Showing 9 changed files with 46,926 additions and 46,841 deletions.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ the file is a comma-separated values file with one or more columns:
* `#did`: the dead identifier
* `when`: when the identifier stopped being used
* `nextofkin`: the identifier that replaces the identifier (for some reason)
* `contributor`: (optional) the ORCID identifier of the curator for the death/replacement

This model is incomplete and volatile, but applies to Release 1.

Expand Down
Loading

0 comments on commit 20dc268

Please sign in to comment.