Feature: Support multilangual search #94
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ields for values in other languages
syphax-bouazzouni
force-pushed
the
feature/support-multi-langual-search
branch
from
766108a to
cba5930
Compare
|
This PR to work in the server we need to update the server SOLR |
syphax-bouazzouni
added a commit
that referenced
this pull request
Sep 5, 2023
…ngual-search Feature: Support multilangual search
syphax-bouazzouni
added a commit
that referenced
this pull request
Sep 5, 2023
…ngual-search Feature: Support multilangual search
syphax-bouazzouni
added a commit
that referenced
this pull request
Dec 5, 2023
#96) * fix json serializer nil exception (#79) if hash was missing @context key the json serializer throwed a nil exception * Merge pull request #87 from ontoportal-lirmm/fix/achived-submission-space-optimization Fix: Archived submission space optimization * Merge pull request #71 from ontoportal-lirmm/feature/support-multilingual-read-one-language-from-request-parameter Feature: Support multilingual - Add read one language from request parameter tests * Merge pull request #73 from ontoportal-lirmm/feature/add-request-language-to-context Feature/add request language to context * fix json serializer nil exception (#79) if hash was missing @context key the json serializer throwed a nil exception * Merge pull request #83 from ontoportal-lirmm/feature/add-request-language-to-context Feature: add request language to context follow up * Merge pull request #93 from ontoportal-lirmm/support-multi-lang-part-02 Feature: Add request all languages unit test * Merge pull request #94 from ontoportal-lirmm/feature/support-multi-langual-search Feature: Support multilangual search --------- Co-authored-by: Raimi Solorzano Niederhausen <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
This PR update and implement the multi-lingual search. With it, we can now for these three properties of Classes/Concepts: prefLabel, synonym, and definition.
This was made possible by doing two main things;
Update our SOLR Schema to have dynamic fields: to save all the values in different languages. For example, a resource that has labels in French, English and not tagged, we will have in SOLR the fields
prefLabel_frfor those in French,prefLabel_enfor those in English,prefLabel_nonefor those with no languages, and lastlyprefLabelfor all the values in all languages mixed see 0c4ba8c).Those dynamic fields will let us do requests like this
.search(prefLabel_fr:Plante)to search only the label in French. And.search(prefLabel:Plante)to search in all the languages.Requesting and indexing all the languages: The work done in Feature: Add request all languages unit test #93 let us fetch all the data in all the languages when we ask for it. Those requested data can then be indexed and saved in their corresponding language dynamic field. ( dynamic fields are created only if values exist in that language, creating only the fields needed) (7aaf942)
Changes