Merge pull request #118 from vitrivr/fix/cosineDistance

Changed cosine similarity to cosine distance
vitrivr · May 8, 2022 · 83a6c47 · 83a6c47
2 parents ce72ef8 + 96051c6
commit 83a6c47
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/...tlin/org/vitrivr/cottontail/core/queries/functions/math/distance/binary/CosineDistance.kt b/...tlin/org/vitrivr/cottontail/core/queries/functions/math/distance/binary/CosineDistance.kt
@@ -71,7 +71,7 @@ sealed class CosineDistance<T : VectorValue<*>>(type: Types.Vector<T,*>): Vector
                 normq += query.data[i].pow(2)
                 normv += probing.data[i].pow(2)
             }
-            return DoubleValue(dotp / (sqrt(normq) * sqrt(normv)))
+            return DoubleValue(1.0 - (dotp / (sqrt(normq) * sqrt(normv))))
         }
         override fun copy(d: Int) = DoubleVector(Types.DoubleVector(d))
     }
@@ -92,7 +92,7 @@ sealed class CosineDistance<T : VectorValue<*>>(type: Types.Vector<T,*>): Vector
                 normq += query.data[i].pow(2)
                 normv += probing.data[i].pow(2)
             }
-            return DoubleValue(dotp / (sqrt(normq) * sqrt(normv)))
+            return DoubleValue(1.0 - (dotp / (sqrt(normq) * sqrt(normv))))
         }
         override fun copy(d: Int) = FloatVector(Types.FloatVector(d))
     }
@@ -113,7 +113,7 @@ sealed class CosineDistance<T : VectorValue<*>>(type: Types.Vector<T,*>): Vector
                 normq += query.data[i].toDouble().pow(2)
                 normv += probing.data[i].toDouble().pow(2)
             }
-            return DoubleValue(dotp / (sqrt(normq) * sqrt(normv)))
+            return DoubleValue(1L - (dotp / (sqrt(normq) * sqrt(normv))))
         }
         override fun copy(d: Int) = LongVector(Types.LongVector(d))
     }
@@ -134,7 +134,7 @@ sealed class CosineDistance<T : VectorValue<*>>(type: Types.Vector<T,*>): Vector
                 normq += query.data[i].toDouble().pow(2)
                 normv += probing.data[i].toDouble().pow(2)
             }
-            return DoubleValue(dotp / (sqrt(normq) * sqrt(normv)))
+            return DoubleValue(1 - (dotp / (sqrt(normq) * sqrt(normv))))
         }
         override fun copy(d: Int) = IntVector(Types.IntVector(d))
     }