Fix style errors

equinor · Jul 7, 2023 · 7f2f740 · 7f2f740
1 parent e8dcd5e
commit 7f2f740
Show file tree

Hide file tree

Showing 9 changed files with 12 additions and 11 deletions.
diff --git a/semeio/communication/semeio_script.py b/semeio/communication/semeio_script.py
@@ -55,6 +55,8 @@ class SemeioScript(ErtScript):
     it forwards log statements to the reporter as well.
     """
 
+    # pylint: disable=abstract-method
+
     def __init__(self, ert, storage, ensemble=None):
         super().__init__(ert, storage, ensemble=ensemble)
         self.facade = LibresFacade(ert)

diff --git a/semeio/workflows/ahm_analysis/ahmanalysis.py b/semeio/workflows/ahm_analysis/ahmanalysis.py
@@ -110,8 +110,7 @@ def run(
         group_by="data_key",
         output_dir=None,
     ):
-        # pylint: disable=method-hidden
-        # pylint: disable=too-many-statements
+        # pylint: disable=method-hidden, too-many-statements, arguments-differ
         # (SemeioScript wraps this run method)
 
         # pylint: disable=too-many-locals

diff --git a/semeio/workflows/correlated_observations_scaling/cos.py b/semeio/workflows/correlated_observations_scaling/cos.py
@@ -134,7 +134,7 @@ def _get_example(config_example):
 
 class CorrelatedObservationsScalingJob(SemeioScript):
     def run(self, job_configuration):
-        # pylint: disable=method-hidden
+        # pylint: disable=method-hidden, arguments-differ
         # (SemeioScript wraps this run method)
         user_config = load_yaml(job_configuration)
         user_config = _insert_default_group(user_config)

diff --git a/semeio/workflows/csv_export2/csv_export2.py b/semeio/workflows/csv_export2/csv_export2.py
@@ -95,7 +95,7 @@ def csv_exporter(runpathfile, time_index, outputfile, column_keys=None):
 
 
 class CsvExport2Job(ErtScript):
-    def run(self, *args):
+    def run(self, *args, **_):
         # pylint: disable=no-self-use
         main(args)
 

diff --git a/semeio/workflows/localisation/local_config_script.py b/semeio/workflows/localisation/local_config_script.py
@@ -6,7 +6,7 @@
 
 
 class LocalisationConfigJob(SemeioScript):
-    def run(self, *args):
+    def run(self, *args, **_):
         # pylint: disable=method-hidden
         # (SemeioScript wraps this run method)
         ert = self.ert()

diff --git a/semeio/workflows/misfit_preprocessor/misfit_preprocessor.py b/semeio/workflows/misfit_preprocessor/misfit_preprocessor.py
@@ -16,7 +16,7 @@
 
 class MisfitPreprocessorJob(SemeioScript):
     # pylint: disable=method-hidden
-    def run(self, *args):
+    def run(self, *args, **_):
         config_record = _fetch_config_record(args)
         observations = list(self.facade.get_observations().obs_vectors.keys())
         config = assemble_config(config_record, observations)

diff --git a/semeio/workflows/spearman_correlation_job/spearman_correlation.py b/semeio/workflows/spearman_correlation_job/spearman_correlation.py
@@ -13,7 +13,7 @@
 
 
 class SpearmanCorrelationJob(SemeioScript):
-    def run(self, *args):
+    def run(self, *args, **_):
         # pylint: disable=method-hidden
         # (SemeioScript wraps this run method)
 

diff --git a/tests/communication/test-data/logging_test_workflow_job.py b/tests/communication/test-data/logging_test_workflow_job.py
@@ -1,12 +1,12 @@
 import logging
 from threading import Thread
 
-from semeio.communication import SemeioScript
+from semeio.communication import SemeioScript  # pylint: disable=import-error
 
 
 class TestWorkflowJob(SemeioScript):
-    # pylint: disable=method-hidden
-    def run(self, *args):
+    # pylint: disable=method-hidden, too-few-public-methods
+    def run(self, *args, **_):
         # pylint: disable=unused-argument
         self.reporter.publish("test_data", list(range(10)))
 

diff --git a/tests/communication/unit/test_semeio_script.py b/tests/communication/unit/test_semeio_script.py
@@ -8,7 +8,7 @@
 
 from semeio.communication import SEMEIOSCRIPT_LOG_FILE, SemeioScript, semeio_script
 
-# pylint: disable=method-hidden,no-self-use
+# pylint: disable=method-hidden,no-self-use, arguments-differ
 
 
 def _ert_mock(monkeypatch, ensemble_path="storage", user_case_name="case_name"):