Skip to content

Commit d7cc153

Browse files
cr feedback
1 parent eb2c822 commit d7cc153

File tree

8 files changed

+43
-43
lines changed

8 files changed

+43
-43
lines changed

docs/reference/query-languages/esql/_snippets/functions/description/v_hamming.md

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

docs/reference/query-languages/esql/_snippets/functions/examples/v_hamming.md

Lines changed: 10 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

docs/reference/query-languages/esql/_snippets/functions/layout/v_hamming.md

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

docs/reference/query-languages/esql/_snippets/functions/parameters/v_hamming.md

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

docs/reference/query-languages/esql/kibana/definition/functions/v_hamming.json

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

docs/reference/query-languages/esql/kibana/docs/functions/v_hamming.md

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

x-pack/plugin/esql/qa/testFixtures/src/main/resources/vector-hamming.csv-spec

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -13,17 +13,17 @@
1313
;
1414

1515
// tag::vector-hamming-result[]
16-
color:text | similarity:double
17-
cyan | 1.0
18-
azure | 0.8333333134651184
19-
blue | 0.6666666865348816
20-
honeydew | 0.6666666865348816
21-
lime | 0.6666666865348816
22-
mint cream | 0.6666666865348816
23-
white | 0.6666666865348816
24-
thistle | 0.625
25-
lavender | 0.5833333134651184
26-
aqua marine | 0.5416666865348816
16+
color:text | similarity:double
17+
red | 24.0
18+
orange | 20.0
19+
gold | 18.0
20+
indigo | 18.0
21+
bisque | 17.0
22+
maroon | 17.0
23+
pink | 17.0
24+
salmon | 17.0
25+
black | 16.0
26+
firebrick | 16.0
2727
// end::vector-hamming-result[]
2828
;
2929

@@ -37,17 +37,17 @@ from colors
3737
| keep color, score
3838
;
3939

40-
color:text | score:double
41-
cyan | 1.5
42-
azure | 1.417
43-
blue | 1.333
44-
honeydew | 1.333
45-
lime | 1.333
46-
mint cream | 1.333
47-
white | 1.333
48-
thistle | 1.313
49-
lavender | 1.292
50-
aqua marine | 1.271
40+
color:text | score:double
41+
red | 13.0
42+
orange | 11.0
43+
gold | 10.0
44+
indigo | 10.0
45+
bisque | 9.5
46+
maroon | 9.5
47+
pink | 9.5
48+
salmon | 9.5
49+
black | 9.0
50+
firebrick | 9.0
5151
;
5252

5353
similarityWithLiteralVectors
@@ -59,7 +59,7 @@ row a = 1
5959
;
6060

6161
similarity:double
62-
0.833
62+
4.0
6363
;
6464

6565
similarityWithStats
@@ -71,7 +71,7 @@ from colors
7171
;
7272

7373
avg:double | min:double | max:double
74-
0.445 | 0.0 | 1.0
74+
13.322 | 0.0 | 24.0
7575
;
7676

7777
similarityWithNull

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/vector/Hamming.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,12 @@ public Hamming(
3939
@Param(
4040
name = "left",
4141
type = { "dense_vector" },
42-
description = "first dense_vector to calculate hamming distance between"
42+
description = "First dense_vector to use to calculate the Hamming distance"
4343
) Expression left,
4444
@Param(
4545
name = "right",
4646
type = { "dense_vector" },
47-
description = "second dense_vector to calculate hamming distance between"
47+
description = "Second dense_vector to use to calculate the Hamming distance"
4848
) Expression right
4949
) {
5050
super(source, left, right);
@@ -83,6 +83,6 @@ public static float calculateSimilarity(float[] leftScratch, float[] rightScratc
8383
for (int i = 0; i < leftScratch.length; i++) {
8484
b[i] = (byte) rightScratch[i];
8585
}
86-
return ((a.length * Byte.SIZE) - VectorUtil.xorBitCount(a, b)) / (float) (a.length * Byte.SIZE);
86+
return VectorUtil.xorBitCount(a, b);
8787
}
8888
}

0 commit comments

Comments
 (0)