Tue, Apr 16, 2019 at 11:16:26AM CEST, jtluka(a)redhat.com wrote:
Tue, Apr 16, 2019 at 10:57:17AM CEST, olichtne(a)redhat.com wrote:
>From: Ondrej Lichtner <olichtne(a)redhat.com>
>
>The function is generic enough and useful enough to move it into the
>lnst.RecipeCommon.Perf.Results module instead of having it as a helper
>method for flow average evaluation.
>
>Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com>
>---
> .../Evaluators/BaselineFlowAverageEvaluator.py | 17 +++++++----------
> lnst/RecipeCommon/Perf/Results.py | 5 +++++
> 2 files changed, 12 insertions(+), 10 deletions(-)
>
>diff --git a/lnst/RecipeCommon/Perf/Evaluators/BaselineFlowAverageEvaluator.py
b/lnst/RecipeCommon/Perf/Evaluators/BaselineFlowAverageEvaluator.py
>index 3f49ab0..ae53bed 100644
>--- a/lnst/RecipeCommon/Perf/Evaluators/BaselineFlowAverageEvaluator.py
>+++ b/lnst/RecipeCommon/Perf/Evaluators/BaselineFlowAverageEvaluator.py
>@@ -6,6 +6,7 @@ from ..Measurements.BaseFlowMeasurement import (
> FlowMeasurementResults,
> AggregatedFlowMeasurementResults,
> )
>+from ..Results import result_averages_difference
^^^^^^^^^
Do you mind adding full path of the import? Besides that the patch looks ok.
-Jan
Sorry for noise, this is fixed in a follwup patch.