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

v1.2.2 #257

Merged
merged 13 commits into from
Jun 5, 2024
Merged

v1.2.2 #257

merged 13 commits into from
Jun 5, 2024

Conversation

vincerubinetti
Copy link
Collaborator

@vincerubinetti vincerubinetti commented Apr 22, 2024

Closes #250
Closes #256
Closes #260
Closes #258

  • adds affiliation to member portrait component
  • simplify portrait component code/css
  • make tag de-dupe behavior the same as search de-dupe. normalize to lower-kebab-case.
  • expand list of manubot-supported id types for falling back to orcid api details
  • change order and type of preferred ids from orcid

Copy link

github-actions bot commented Apr 22, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-06-05 20:27 UTC

@vincerubinetti vincerubinetti removed the request for review from falquaddoomi May 7, 2024 19:45
@vincerubinetti
Copy link
Collaborator Author

@falquaddoomi Can you take a look at this? The main changes of note are in orcid.py, and the more possibly questionable changes involve issue #258. Summary: not sure which ORCID API fields to use. It returns a lot of confusing and duplicate fields, their documentation is insufficient, and there's basically no one I can contact to clarify. We have one data point (Andrew Su) where these changes match their expectations, and one data point (Casey Greene) where these changes don't produce any different results.

@falquaddoomi
Copy link
Contributor

falquaddoomi commented May 30, 2024

Hey @vincerubinetti; agreed, the ORCID API docs leave a lot to be desired.

Regarding issue #258, I think I was able to replicate the behavior Andrew reported with a publication in my ORCID account that has three sources: changing the preferred source changed the order of the elements in work-summary for the publication's group, as well as changing the display-index for that source. There's a bit from the ORCID API tutorial on the display index (https://info.orcid.org/ufaqs/how-are-items-grouped-together-in-an-orcid-record/):

Our API provides support for this in the XSD. Each item has a display index attribute which indicates its rank within its group. The highest display index is the preferred item selected by the researcher, items added via the API which have not been ranked by the researcher have a display index of 0. The display index also determines the work order when reading the ORCID record.

I don't know if that's enough to resolve that issue, but I hope it helps.

@vincerubinetti
Copy link
Collaborator Author

That's great, thanks for testing that out. None of my ORCID entries had multiple work summaries to test that with. With that extra data point, I feel confident enough that the api will do what it says (user-specified preferred sources become the highest display-index, and api returns in display-index order) and that this is the right decision.

Copy link
Contributor

@falquaddoomi falquaddoomi left a comment

Choose a reason for hiding this comment

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

Sorry for the delay; I made some comments on the Python code. All the other changes seemed straightforward to me.

_cite/plugins/orcid.py Outdated Show resolved Hide resolved
_cite/plugins/orcid.py Show resolved Hide resolved
_cite/plugins/orcid.py Show resolved Hide resolved
@vincerubinetti
Copy link
Collaborator Author

@andrewsu Do you have time to give this code a quick look and these set of issues one last think?

@vincerubinetti vincerubinetti merged commit 5afb459 into main Jun 5, 2024
4 checks passed
@vincerubinetti vincerubinetti deleted the v1.2.2 branch June 5, 2024 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants