lab-website-template icon indicating copy to clipboard operation
lab-website-template copied to clipboard

v1.2.2

Open vincerubinetti opened this issue 1 year ago • 1 comments
trafficstars

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

vincerubinetti avatar Apr 22 '24 20:04 vincerubinetti

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

github-actions[bot] avatar Apr 22 '24 20:04 github-actions[bot]

@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 https://github.com/greenelab/lab-website-template/issues/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.

vincerubinetti avatar May 29 '24 21:05 vincerubinetti

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.

falquaddoomi avatar May 30 '24 15:05 falquaddoomi

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.

vincerubinetti avatar May 30 '24 16:05 vincerubinetti

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

vincerubinetti avatar May 30 '24 17:05 vincerubinetti