Use a weighted search #122

extracts · 2023-11-16T19:42:58Z

PR für #37

src/Query.php

src/Solr/Solarium/Adapter.php

…(using the eDisMax query parser) works as expected

j3nsch

Vielen Dank! Die Frage der Konfiguration muss noch geklärt und ein paar Kleinigkeiten angepasst werden.

src/Query.php

src/Solr/Solarium/Adapter.php

test/Solr/Solarium/AdapterSearchingTest.php

test/TestAsset/DocumentBasedTestCase.php

src/Solr/Solarium/Adapter.php

…ue or false, not null

…elds influences the result scores

…e or false (but not null)

…ights influences the order of results

…ation opions if defined

…ng for missing or equal boost factors to a separate test

extracts · 2023-11-23T20:44:57Z

@j3nsch Dieser PR ist jetzt funktional & verwendbar.
Die Solr-Parameter zur besseren Gewichtung von zusammenhängenden Wortfolgen ("phrase search") sind jedoch noch nicht implementiert.

…'s ID (and not its title)

…ds() which will read the configuration only once

src/Query.php

…t getWeightedSearch() can load its value from the config if it isn't set yet

…fluent interface

…quest parameter that's used when matching phrases

…eDisMax Solr query parsers

…a; by default, Solr's standard query parser will now only search the "title" field; adopts test accordingly

extracts · 2023-12-03T20:20:01Z

Nach dem Entfernen der "catchall" Felder text & simple aus dem Solr Schema schlägt einer der existierenden Tests fehl (AdapterSearchingTest->testSearchWithDiacritics()).

Die Solr Standard-Suche sucht jetzt standardmässig nur noch im "Titel"-Feld. Für eine Suche über alle Felder benötigt es jetzt die gewichtete Suche.

… default Setting a field's boost factor to 0 (via the "search.simple" config option) will cause documents with matches just in that field to get a score of 0

…ighted search (which searches across all defined fields instead of just the title field)

Merge in support for PublicationState

#37 Adds a test that searches Solr using the eDisMax query parser (WIP)

be3ee4f

extracts requested a review from j3nsch November 16, 2023 19:42

extracts marked this pull request as draft November 16, 2023 19:43

extracts linked an issue Nov 16, 2023 that may be closed by this pull request

Catch-All-Feldsuche durch gewichtete Suche in einzelnen Indexfeldern ersetzen #37

Open

#37 Fix "call to undefined method" error

214c501

j3nsch reviewed Nov 16, 2023

View reviewed changes

src/Query.php Outdated Show resolved Hide resolved

j3nsch reviewed Nov 16, 2023

View reviewed changes

src/Solr/Solarium/Adapter.php Outdated Show resolved Hide resolved

j3nsch reviewed Nov 16, 2023

View reviewed changes

src/Solr/Solarium/Adapter.php Show resolved Hide resolved

extracts added 2 commits November 17, 2023 19:35

#37 Keep Solr-specific terminology & syntax out of Opus\Search\Query

0f387b9

#37 Completes search test which verifies that a weighted Solr search …

105ca56

…(using the eDisMax query parser) works as expected

j3nsch assigned extracts Nov 21, 2023

j3nsch requested changes Nov 21, 2023

View reviewed changes

extracts added 8 commits November 22, 2023 14:15

#37 Default values for getters that return a boolean value must be tr…

b0b22a4

…ue or false, not null

#37 The weighted search test now shows that searching with boosted fi…

b290e62

…elds influences the result scores

#37 Adopt tests to Opus\Search\Query->getUnion() returning either tru…

abe0045

…e or false (but not null)

#37 The weighted search test now also verifies that swapping field we…

527e42c

…ights influences the order of results

#37 Default to the "search.weightedSearch" & "search.simple" configur…

541bc43

…ation opions if defined

#37 Now checks the sort order of weighted search results; moves testi…

c78afc7

…ng for missing or equal boost factors to a separate test

#37 Reuse test documents between weighted search tests

66c06a1

#37 Fixes a namespace conflict

c622821

extracts marked this pull request as ready for review November 23, 2023 20:32

extracts added 2 commits November 24, 2023 17:36

#37 Verify the sort order of weighted search results via the document…

42b9eb0

…'s ID (and not its title)

#37 Implements explicit getters getWeightedSearch() & getWeightedFiel…

f6d7469

…ds() which will read the configuration only once

extracts commented Nov 24, 2023

View reviewed changes

src/Query.php Outdated Show resolved Hide resolved

extracts added 5 commits November 29, 2023 16:14

#37 Removes the weightedsearch key from the initial data array so tha…

ad067cb

…t getWeightedSearch() can load its value from the config if it isn't set yet

#37 Fix missing return statement in setWeightedSearch() which uses a …

e0593c0

…fluent interface

#37 Adds a weight multiplier to generate a value for the Solr "pf" re…

357b053

…quest parameter that's used when matching phrases

#37 Adds a test that compares the search behaviour of the standard & …

54d1760

…eDisMax Solr query parsers

#37 Fix coding style

a47e997

extracts added 3 commits December 1, 2023 18:50

#37 Replaces redundant boiler plate code with separate helper methods

efbba3b

#37 More (and more granular) tests that test weighted search behavior

e635d6f

#37 Removes the catchall fields "text" & "simple" from the Solr schem…

0d51ed8

…a; by default, Solr's standard query parser will now only search the "title" field; adopts test accordingly

extracts and others added 5 commits December 4, 2023 19:55

#37 When searching Solr, matches with a score of 0 are now ignored by…

4577fcf

… default Setting a field's boost factor to 0 (via the "search.simple" config option) will cause documents with matches just in that field to get a score of 0

#37 Adopts a test that searches the author field so that it uses a we…

9e165b1

…ighted search (which searches across all defined fields instead of just the title field)

Merge branch '4.8.1' into weightedSearch37

8a08dff

Merge pull request #130 from OPUS4/weightedSearch37tmp

6066e6a

Merge in support for PublicationState

#131 Added test for advanced search

cf8cb36

Base automatically changed from 4.8.1 to master October 15, 2024 02:40

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Use a weighted search #122

Use a weighted search #122

extracts commented Nov 16, 2023

j3nsch left a comment

extracts commented Nov 23, 2023

extracts commented Dec 3, 2023 •

edited

Loading

Use a weighted search #122

Are you sure you want to change the base?

Use a weighted search #122

Conversation

extracts commented Nov 16, 2023

j3nsch left a comment

Choose a reason for hiding this comment

extracts commented Nov 23, 2023

extracts commented Dec 3, 2023 • edited Loading

extracts commented Dec 3, 2023 •

edited

Loading