Fix based on review

pyscal/__init__.py

@@ -19,10 +19,11 @@ def getLogger_pyscal(
     Logging output is by default split by logging levels (split between WARNING and
     ERROR) to stdout and stderr, each log occurs in only one of the streams.
     Args:
-        module_name: A suggested name for the logger, usually __name__ should be supplied
-        args_dict: Dictionary with contents from the argparse namespace object containing
-        only keys "output", "verbose" and "debug".
-    """  # noqa: E501
+        module_name: A suggested name for the logger, usually __name__ should be
+        supplied
+        args_dict: Dictionary with contents from the argparse namespace object
+        containing only keys "output", "verbose" and "debug".
+    """
     logger = logging.getLogger(module_name)
     if len(logger.handlers) != 0:
         return logger

pyscal/factory.py

@@ -154,7 +154,7 @@ class PyscalFactory(object):
     @staticmethod
     def create_water_oil(
         params: Optional[Dict[str, float]] = None, fast: bool = False
-    ) -> WaterOil:  # noqa
+    ) -> WaterOil:
         """Create a WaterOil object from a dictionary of parameters.
 
         Parameterization (Corey/LET) is inferred from presence
@@ -1208,7 +1208,7 @@ def filter_nan_from_dict(params: dict) -> dict:
     """
     cleaned_params = {}
     for key, value in params.items():
-        if (isinstance(value, str)) or (not np.isnan(value)):
+        if isinstance(value, str) or not np.isnan(value):
             cleaned_params[key] = value
     return cleaned_params
 

tests/test_gaswater.py

@@ -116,7 +116,7 @@ def test_gaswater_corey1(nw, ng):
 
 
 @given(st.floats(), st.floats(), st.floats(), st.floats(), st.floats())
-def test_gaswater_let1(l, e, t, krwend, krwmax):  # noqa: E741
+def test_gaswater_let1(l, e, t, krwend, krwmax):
     """Test random LET parameters"""
     gaswater = GasWater()
     try: