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

Facet sorting by year and author not working #572

Closed
1 task
astrochun opened this issue Feb 27, 2024 · 3 comments · Fixed by #574
Closed
1 task

Facet sorting by year and author not working #572

astrochun opened this issue Feb 27, 2024 · 3 comments · Fixed by #574
Assignees

Comments

@astrochun
Copy link

astrochun commented Feb 27, 2024

Recently tried to sort the PPPL results of datasets in Discovery by year and authorship and the results were inconsistent with expectations.

See screenshots:

By year:
Screen Shot 2024-02-27 at 7 26 25 AM

By author:
Screen Shot 2024-02-27 at 7 26 44 AM

Might be related to the home page show. These items show both describe and dataspace. PPPL content has been migrated, so we may not to worry about the dpsace content

Acceptance criteria:

  • Utilize the new sorting field in the blacklight config may be something like "created_timestamp"
@carolyncole
Copy link
Member

@carolyncole carolyncole self-assigned this Feb 28, 2024
@carolyncole
Copy link
Member

@astrochun The way the current Author sort is working is that the authors of the paper are sorted in alphabetical order and then the one earliest in the alphabet is utilized. So for a paper that has "Smith,Jane" as the first author and "Doe, John" as the second author, because Doe is earlier in the alphabet it is utilized for sorting. I would assume that we should really just be sorting by the first author?

carolyncole added a commit that referenced this issue Feb 29, 2024
The sort field for year was not defined by the PDC indexer, which caused the sort to be random
The Author sort was the sorting by the alpabetically first of any of the authors, which made the sort seem random
fixes #572
carolyncole added a commit that referenced this issue Feb 29, 2024
The sort field for year was not defined by the PDC indexer, which caused the sort to be random
The Author sort was the sorting by the alpabetically first of any of the authors, which made the sort seem random
fixes #572
carolyncole added a commit that referenced this issue Feb 29, 2024
The sort field for year was not defined by the PDC indexer, which caused the sort to be random
The Author sort was the sorting by the alpabetically first of any of the authors, which made the sort seem random
fixes #572
carolyncole added a commit that referenced this issue Mar 1, 2024
The sort field for year was not defined by the PDC indexer, which caused the sort to be random
The Author sort was the sorting by the alpabetically first of any of the authors, which made the sort seem random
fixes #572
carolyncole added a commit that referenced this issue Mar 1, 2024
The sort field for year was not defined by the PDC indexer, which caused the sort to be random
The Author sort was the sorting by the alpabetically first of any of the authors, which made the sort seem random
fixes #572
carolyncole added a commit that referenced this issue Mar 1, 2024
The sort field for year was not defined by the PDC indexer, which caused the sort to be random
The Author sort was the sorting by the alpabetically first of any of the authors, which made the sort seem random
fixes #572
carolyncole added a commit that referenced this issue Mar 1, 2024
The sort field for year was not defined by the PDC indexer, which caused the sort to be random
The Author sort was the sorting by the alpabetically first of any of the authors, which made the sort seem random
fixes #572
carolyncole added a commit that referenced this issue Mar 1, 2024
The sort field for year was not defined by the PDC indexer, which caused the sort to be random
The Author sort was the sorting by the alpabetically first of any of the authors, which made the sort seem random
fixes #572
carolyncole added a commit that referenced this issue Mar 1, 2024
The sort field for year was not defined by the PDC indexer, which caused the sort to be random
The Author sort was the sorting by the alpabetically first of any of the authors, which made the sort seem random
fixes #572
carolyncole added a commit that referenced this issue Mar 1, 2024
The sort field for year was not defined by the PDC indexer, which caused the sort to be random
The Author sort was the sorting by the alpabetically first of any of the authors, which made the sort seem random
fixes #572
carolyncole added a commit that referenced this issue Mar 4, 2024
The sort field for year was not defined by the PDC indexer, which caused the sort to be random
The Author sort was the sorting by the alpabetically first of any of the authors, which made the sort seem random
fixes #572
@astrochun
Copy link
Author

@astrochun The way the current Author sort is working is that the authors of the paper are sorted in alphabetical order and then the one earliest in the alphabet is utilized. So for a paper that has "Smith,Jane" as the first author and "Doe, John" as the second author, because Doe is earlier in the alphabet it is utilized for sorting. I would assume that we should really just be sorting by the first author?

Understood so then why is the example I provided shows Mollen before Gan? There should be some with last name starting in A's.

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

Successfully merging a pull request may close this issue.

2 participants