From ea654523b71d2bafc941f8b95d06909324e3c902 Mon Sep 17 00:00:00 2001 From: Vratko Polak Date: Fri, 30 Sep 2022 14:24:53 +0200 Subject: fix(soak): scale also rarity Without this, far-off samples are considered too important leading to wrong integration. This should fix the symptom of secondary stats being too different. Change-Id: I66496ac25da95eae1f83154f32eb56f941cb20ce Signed-off-by: Vratko Polak (cherry picked from commit d772d5346cf1c4802bda63e4faf008222b6c0cf0) --- resources/libraries/python/PLRsearch/Integrator.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/resources/libraries/python/PLRsearch/Integrator.py b/resources/libraries/python/PLRsearch/Integrator.py index a7a59391ed..7f118db00d 100644 --- a/resources/libraries/python/PLRsearch/Integrator.py +++ b/resources/libraries/python/PLRsearch/Integrator.py @@ -1,4 +1,4 @@ -# Copyright (c) 2021 Cisco and/or its affiliates. +# Copyright (c) 2022 Cisco and/or its affiliates. # Licensed under the Apache License, Version 2.0 (the "License"); # you may not use this file except in compliance with the License. # You may obtain a copy of the License at: @@ -253,7 +253,7 @@ def estimate_nd(communication_pipe, scale_coeff=8.0, trace_enabled=False): ) # The code above looked at weight (not importance). # The code below looks at importance (not weight). - log_rarity = param_distance / 2.0 + log_rarity = param_distance / 2.0 / scale_coeff trace(u"log_rarity", log_rarity) log_importance = log_weight + log_rarity trace(u"log_importance", log_importance) -- cgit 1.2.3-korg