aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVratko Polak <vrpolak@cisco.com>2022-09-30 14:24:53 +0200
committerVratko Polak <vrpolak@cisco.com>2022-09-30 15:52:00 +0200
commitd772d5346cf1c4802bda63e4faf008222b6c0cf0 (patch)
tree9124320086b3c10313807b1ab0f5a1de477fca55
parent7df86999bfaee9e5a0fa0c8ee6f5bc897343de9a (diff)
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 <vrpolak@cisco.com>
-rw-r--r--resources/libraries/python/PLRsearch/Integrator.py4
1 files 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)