Browse Source

[mod] metrics: add to pylint and justify indentations

Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
Markus Heiser 4 years ago
parent
commit
c54bf42cb9
2 changed files with 19 additions and 16 deletions
  1. 2 1
      Makefile
  2. 17 15
      searx/metrics/__init__.py

+ 2 - 1
Makefile

@@ -199,7 +199,8 @@ PYLINT_FILES=\
 	searx/engines/yahoo_news.py \
 	searx/engines/yahoo_news.py \
 	searx/engines/apkmirror.py \
 	searx/engines/apkmirror.py \
 	searx/engines/artic.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
 test.pylint: pyenvinstall
 	$(call cmd,pylint,$(PYLINT_FILES))
 	$(call cmd,pylint,$(PYLINT_FILES))

+ 17 - 15
searx/metrics/__init__.py

@@ -1,4 +1,5 @@
 # SPDX-License-Identifier: AGPL-3.0-or-later
 # SPDX-License-Identifier: AGPL-3.0-or-later
+# pylint: disable=missing-module-docstring, missing-function-docstring
 
 
 import typing
 import typing
 import math
 import math
@@ -63,7 +64,7 @@ def initialize(engine_names=None):
     """
     """
     Initialize metrics
     Initialize metrics
     """
     """
-    global counter_storage, histogram_storage
+    global counter_storage, histogram_storage # pylint: disable=global-statement
 
 
     counter_storage = CounterStorage()
     counter_storage = CounterStorage()
     histogram_storage = HistogramStorage()
     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_time_total = max(time_total, max_time_total or 0)
         max_result_count = max(result_count, max_result_count 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 {
     return {
         'time': list_time,
         'time': list_time,