Skip to content

Commit dff2a71

Browse files
owaiskazi19ryanbogan
authored andcommitted
Fixed IT for Zscore (opensearch-project#1242)
Signed-off-by: Owais <[email protected]>
1 parent 1db53d0 commit dff2a71

File tree

2 files changed

+46
-48
lines changed

2 files changed

+46
-48
lines changed

src/test/java/org/opensearch/neuralsearch/highlight/SemanticHighlighterIT.java

Lines changed: 45 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -99,11 +99,11 @@ public void testQueriesWithSemanticHighlighter() throws Exception {
9999
*/
100100
MatchQueryBuilder matchQuery = new MatchQueryBuilder(TEST_TEXT_FIELD_NAME, TEST_QUERY_TEXT);
101101
Map<String, Object> searchResponse = searchWithSemanticHighlighter(
102-
TEST_BASIC_INDEX_NAME,
103-
matchQuery,
104-
10,
105-
TEST_TEXT_FIELD_NAME,
106-
sentenceHighlightingModelId
102+
TEST_BASIC_INDEX_NAME,
103+
matchQuery,
104+
10,
105+
TEST_TEXT_FIELD_NAME,
106+
sentenceHighlightingModelId
107107
);
108108
verifyHighlightResults(searchResponse, TEST_QUERY_TEXT);
109109

@@ -128,11 +128,11 @@ public void testQueriesWithSemanticHighlighter() throws Exception {
128128
*/
129129
TermQueryBuilder termQuery = new TermQueryBuilder(TEST_TEXT_FIELD_NAME, "intelligence");
130130
searchResponse = searchWithSemanticHighlighter(
131-
TEST_BASIC_INDEX_NAME,
132-
termQuery,
133-
10,
134-
TEST_TEXT_FIELD_NAME,
135-
sentenceHighlightingModelId
131+
TEST_BASIC_INDEX_NAME,
132+
termQuery,
133+
10,
134+
TEST_TEXT_FIELD_NAME,
135+
sentenceHighlightingModelId
136136
);
137137
verifyHighlightResults(searchResponse, "intelligence");
138138

@@ -161,16 +161,15 @@ public void testQueriesWithSemanticHighlighter() throws Exception {
161161
* }
162162
* }
163163
*/
164-
BoolQueryBuilder boolQuery = new BoolQueryBuilder()
165-
.must(new MatchQueryBuilder(TEST_TEXT_FIELD_NAME, "artificial"))
166-
.must(new MatchQueryBuilder(TEST_TEXT_FIELD_NAME, "intelligence"))
167-
.should(new MatchQueryBuilder(TEST_TEXT_FIELD_NAME, "learning"));
164+
BoolQueryBuilder boolQuery = new BoolQueryBuilder().must(new MatchQueryBuilder(TEST_TEXT_FIELD_NAME, "artificial"))
165+
.must(new MatchQueryBuilder(TEST_TEXT_FIELD_NAME, "intelligence"))
166+
.should(new MatchQueryBuilder(TEST_TEXT_FIELD_NAME, "learning"));
168167
searchResponse = searchWithSemanticHighlighter(
169-
TEST_BASIC_INDEX_NAME,
170-
boolQuery,
171-
10,
172-
TEST_TEXT_FIELD_NAME,
173-
sentenceHighlightingModelId
168+
TEST_BASIC_INDEX_NAME,
169+
boolQuery,
170+
10,
171+
TEST_TEXT_FIELD_NAME,
172+
sentenceHighlightingModelId
174173
);
175174
verifyHighlightResults(searchResponse, "artificial intelligence");
176175

@@ -194,14 +193,13 @@ public void testQueriesWithSemanticHighlighter() throws Exception {
194193
* }
195194
* }
196195
*/
197-
QueryStringQueryBuilder queryString = new QueryStringQueryBuilder("artificial AND intelligence")
198-
.defaultField(TEST_TEXT_FIELD_NAME);
196+
QueryStringQueryBuilder queryString = new QueryStringQueryBuilder("artificial AND intelligence").defaultField(TEST_TEXT_FIELD_NAME);
199197
searchResponse = searchWithSemanticHighlighter(
200-
TEST_BASIC_INDEX_NAME,
201-
queryString,
202-
10,
203-
TEST_TEXT_FIELD_NAME,
204-
sentenceHighlightingModelId
198+
TEST_BASIC_INDEX_NAME,
199+
queryString,
200+
10,
201+
TEST_TEXT_FIELD_NAME,
202+
sentenceHighlightingModelId
205203
);
206204
verifyHighlightResults(searchResponse, "artificial intelligence");
207205

@@ -229,17 +227,17 @@ public void testQueriesWithSemanticHighlighter() throws Exception {
229227
* }
230228
*/
231229
NeuralQueryBuilder neuralQueryBuilder = NeuralQueryBuilder.builder()
232-
.fieldName(TEST_KNN_VECTOR_FIELD_NAME)
233-
.queryText(TEST_QUERY_TEXT)
234-
.modelId(textEmbeddingModelId)
235-
.k(1)
236-
.build();
230+
.fieldName(TEST_KNN_VECTOR_FIELD_NAME)
231+
.queryText(TEST_QUERY_TEXT)
232+
.modelId(textEmbeddingModelId)
233+
.k(1)
234+
.build();
237235
searchResponse = searchWithSemanticHighlighter(
238-
TEST_BASIC_INDEX_NAME,
239-
neuralQueryBuilder,
240-
10,
241-
TEST_TEXT_FIELD_NAME,
242-
sentenceHighlightingModelId
236+
TEST_BASIC_INDEX_NAME,
237+
neuralQueryBuilder,
238+
10,
239+
TEST_TEXT_FIELD_NAME,
240+
sentenceHighlightingModelId
243241
);
244242
verifyHighlightResults(searchResponse, TEST_QUERY_TEXT);
245243

@@ -278,20 +276,20 @@ public void testQueriesWithSemanticHighlighter() throws Exception {
278276
*/
279277
MatchQueryBuilder hybridMatchQuery = new MatchQueryBuilder(TEST_TEXT_FIELD_NAME, TEST_QUERY_TEXT);
280278
NeuralQueryBuilder hybridNeuralQuery = NeuralQueryBuilder.builder()
281-
.fieldName(TEST_KNN_VECTOR_FIELD_NAME)
282-
.queryText("AI applications")
283-
.modelId(textEmbeddingModelId)
284-
.k(1)
285-
.build();
279+
.fieldName(TEST_KNN_VECTOR_FIELD_NAME)
280+
.queryText("AI applications")
281+
.modelId(textEmbeddingModelId)
282+
.k(1)
283+
.build();
286284
SearchSourceBuilder searchSourceBuilder = new SearchSourceBuilder().query(
287-
new BoolQueryBuilder().should(hybridMatchQuery).should(hybridNeuralQuery)
285+
new BoolQueryBuilder().should(hybridMatchQuery).should(hybridNeuralQuery)
288286
);
289287
searchResponse = searchWithSemanticHighlighter(
290-
TEST_BASIC_INDEX_NAME,
291-
searchSourceBuilder.query(),
292-
10,
293-
TEST_TEXT_FIELD_NAME,
294-
sentenceHighlightingModelId
288+
TEST_BASIC_INDEX_NAME,
289+
searchSourceBuilder.query(),
290+
10,
291+
TEST_TEXT_FIELD_NAME,
292+
sentenceHighlightingModelId
295293
);
296294
verifyHighlightResults(searchResponse, "artificial intelligence");
297295
}

src/test/java/org/opensearch/neuralsearch/processor/ScoreNormalizationIT.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,7 @@ public void testZScoreNorm_whenOneShardAndQueryMatches_thenSuccessful() {
292292
Map.of("search_pipeline", SEARCH_PIPELINE)
293293
);
294294

295-
assertHybridSearchResults(searchResponseAsMapArithmeticMean, 5, new float[] { 0.4f, 2.0f });
295+
assertHybridSearchResults(searchResponseAsMapArithmeticMean, 5, new float[] { 0.3f, 2.0f });
296296

297297
deleteSearchPipeline(SEARCH_PIPELINE);
298298

0 commit comments

Comments
 (0)