-
Notifications
You must be signed in to change notification settings - Fork 2
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
Improve full-text searching by expanding reference primary #100
Comments
Horse is a primary name in Equus Caballus, but not labelled as english. This is why it appears, but Ovis does not have this. @clarkepeterf will try using the referencename index to in DEV so it can be reviewed. This include anything in identifedBy with type of name. |
Preliminary deployment in DEV looks good. Propose to also do the following:
|
Per Interactions 4/25
|
Performance test needed before this is deployed. |
@clarkepeterf, #11 (comment) captures related comparisons. |
… term to use setName index field, as that does not exist yet)
Performance tests on 5/9 failed; still determining if issue is with this issue or ML. |
IT meeting 6/21: |
Approved by UAT |
Looks good in prod! Closing. |
Problem Description:
All records (YPM, AAT, Wikidata, GBIF) have the same primary name: Equus Caballus. Most users would search "horse". The issue is raised by art museums, who use Horse as subject heading, and include this in their data under "represents". Discussion surfaced an inconsistency in how simple search is working.
Question: how is simple search working? It is not consistent for this use case.
Expected Behavior/Solution:
Simple search should work such that boxley sheep would retrieve the correct record, even though the primary name is Ovis.
Change simple search to use
referenceName
instead ofreferencePrimaryName
Make the same change for the Name field in Advanced Search.
More detailed description of changes:
referencePrimaryName
field to thereferenceName
field. By search scope, here are the record types that included or excluded from reference fields:name
search terms less theset
search scope switched from their search scope-specific primary name field to their broader name field; for instance, thename
search term in theagent
scope was changed fromagentPrimaryName
toagentName
.Requirements:
referencePrimaryName
Needed for promotion:
If an item on the list is not needed, it should be crossed off but not removed.
Wireframe/Mockup - MikeFeasibility/Team discussion - SarahUAT/LUX Examples:
The searches below should show this record in the results: https://lux.collections.yale.edu/view/visual/b50fb93f-dc1d-4e49-a3b8-acc1cfd4a174
Dependencies/Blocks:
Related Github Issues:
Related links:
Wireframe/Mockup:
Place wireframe/mockup for the proposed solution at end of ticket.
The text was updated successfully, but these errors were encountered: