From c54bf42cb9699501ccbd54a2df0d7fa73be8dcdb Mon Sep 17 00:00:00 2001 From: Markus Heiser Date: Thu, 22 Apr 2021 16:05:39 +0200 Subject: [PATCH] [mod] metrics: add to pylint and justify indentations Signed-off-by: Markus Heiser --- Makefile | 3 ++- searx/metrics/__init__.py | 32 +++++++++++++++++--------------- 2 files changed, 19 insertions(+), 16 deletions(-) diff --git a/Makefile b/Makefile index 166fb65e4..7149437be 100644 --- a/Makefile +++ b/Makefile @@ -199,7 +199,8 @@ PYLINT_FILES=\ searx/engines/yahoo_news.py \ searx/engines/apkmirror.py \ searx/engines/artic.py \ - searx_extra/update/update_external_bangs.py + searx_extra/update/update_external_bangs.py \ + searx/metrics/__init__.py test.pylint: pyenvinstall $(call cmd,pylint,$(PYLINT_FILES)) diff --git a/searx/metrics/__init__.py b/searx/metrics/__init__.py index 063554f1a..9648e6215 100644 --- a/searx/metrics/__init__.py +++ b/searx/metrics/__init__.py @@ -1,4 +1,5 @@ # SPDX-License-Identifier: AGPL-3.0-or-later +# pylint: disable=missing-module-docstring, missing-function-docstring import typing import math @@ -63,7 +64,7 @@ def initialize(engine_names=None): """ Initialize metrics """ - global counter_storage, histogram_storage + global counter_storage, histogram_storage # pylint: disable=global-statement counter_storage = CounterStorage() histogram_storage = HistogramStorage() @@ -164,20 +165,21 @@ def get_engines_stats(engine_list): max_time_total = max(time_total, max_time_total or 0) max_result_count = max(result_count, max_result_count or 0) - list_time.append({'total': round(time_total, 1), - 'total_p80': round(time_total_p80, 1), - 'total_p95': round(time_total_p95, 1), - 'http': round(time_http, 1), - 'http_p80': round(time_http_p80, 1), - 'http_p95': round(time_http_p95, 1), - 'name': engine_name, - 'processing': round(time_total - time_http, 1), - 'processing_p80': round(time_total_p80 - time_http_p80, 1), - 'processing_p95': round(time_total_p95 - time_http_p95, 1), - 'score': score, - 'score_per_result': score_per_result, - 'result_count': result_count, - }) + list_time.append({ + 'total': round(time_total, 1), + 'total_p80': round(time_total_p80, 1), + 'total_p95': round(time_total_p95, 1), + 'http': round(time_http, 1), + 'http_p80': round(time_http_p80, 1), + 'http_p95': round(time_http_p95, 1), + 'name': engine_name, + 'processing': round(time_total - time_http, 1), + 'processing_p80': round(time_total_p80 - time_http_p80, 1), + 'processing_p95': round(time_total_p95 - time_http_p95, 1), + 'score': score, + 'score_per_result': score_per_result, + 'result_count': result_count, + }) return { 'time': list_time,