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

Should ORCIDs be used in the RDF interface where available? #15

Open
jesusbagpuss opened this issue Nov 14, 2018 · 3 comments
Open

Should ORCIDs be used in the RDF interface where available? #15

jesusbagpuss opened this issue Nov 14, 2018 · 3 comments

Comments

@jesusbagpuss
Copy link
Collaborator

By default, EPrints generates a URI based on the person->id, or the person->name (with some extras thrown in): https://github.com/eprints/eprints/blob/v3.3.16/lib/cfg.d/rdf_uri_person.pl

Should this config be changed (redefined in the archive cfg.d) to use $person->{orcid} by default?

The code could be put into e.g. z_orcid_rdf_person.pl.example so it has to be explicitly enabled?

@jesusbagpuss
Copy link
Collaborator Author

NB changing the above would require regenerating the RDF triples.
It would also mean once a triple has been updated, the old URI wouldn't resolve.

An alternative approach (which might actually be more sense) would be to add a new EP_RDF_TRIGGER that replicates this https://github.com/eprints/eprints/blob/v3.3.16/lib/cfg.d/rdf_triples_bibo.pl#L384-L527, but calls a new config method e.g. $c->{rdf}->{person_orcid_uri} instead.

@wfyson
Copy link
Collaborator

wfyson commented Apr 12, 2022

Just doing a bit of long overdue tidying up of issues and pull requests in the ORCID plugins... does https://github.com/eprints/orcid_support_advance/blob/master/cfg/cfg.d/z_orcid_support_advance_rdf_triples.pl fix this issue @jesusbagpuss? It's in the Advance plugin rather than the basic one, but can we close this?

@jesusbagpuss
Copy link
Collaborator Author

@wfyson - just doing an even more overdue tidying of things...
There is an issue with the current rdf-triples solution. I have logged this and proposed a solution/submitted a PR: eprints/orcid_support_advance#80

It feels like the solution in the advance plugin might better live in here - if an EPrints repo is connected to a CRIS system, the advance plugin might not be installed/active, but the RDF would still benefit from having ORCIDs in it?

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

No branches or pull requests

2 participants