FIX: linting

Author: wolfidan

.gitignore

@@ -16,6 +16,7 @@
 *.tmp
 *.vim
 tags
+.vscode
 
 # Compiled source #
 ###################

src/pyart

@@ -1038,15 +1038,16 @@ def _create_cfg_dict(cfgfile):
     try:
         print("- Main config file : {}".format(cfgfile))
         cfg = read_config(cfg["configFile"], cfg=cfg, defaults=DEFAULT_CONFIG["main"])
-        
+
         # Convert loc and prod config files to absolute paths if needed
-        filenames = ["locationConfigFile", 
-        "productConfigFile",
+        filenames = [
+            "locationConfigFile",
+            "productConfigFile",
         ]
         for fname in filenames:
             if not os.path.isabs(cfg[fname]):
-                cfg[fname] = os.path.join(cfg['configpath'], cfg[fname])
-                
+                cfg[fname] = os.path.join(cfg["configpath"], cfg[fname])
+
         print("- Location config file : {}".format(cfg["locationConfigFile"]))
         cfg = read_config(
             cfg["locationConfigFile"], cfg=cfg, defaults=DEFAULT_CONFIG["loc"]
@@ -1183,7 +1184,8 @@ def _create_cfg_dict(cfgfile):
 
     # check whether specified paths are relative or absolute
     # if relative add configpath to the path
-    filenames = ["locationConfigFile", 
+    filenames = [
+        "locationConfigFile",
         "productConfigFile",
         "datapath",
         "iconpath",
@@ -1194,17 +1196,18 @@ def _create_cfg_dict(cfgfile):
         "psrpath",
         "iqpath",
         "satpath",
-        "smnpath"]
-    
+        "smnpath",
+    ]
+
     for fname in filenames:
         if type(cfg[fname]) is list:
             for val in cfg[fname]:
                 if not os.path.isabs(val):
-                    val = os.path.join(cfg['configpath'], val)
+                    val = os.path.join(cfg["configpath"], val)
         elif type(cfg[fname]) is str:
             if not os.path.isabs(cfg[fname]):
-                cfg[fname] = os.path.join(cfg['configpath'], cfg[fname])
-    
+                cfg[fname] = os.path.join(cfg["configpath"], cfg[fname])
+
     # if specified in config, convert coordinates to arrays
     if "RadarPosition" in cfg:
         fltarr_list = ["latitude", "longitude", "altitude"]
@@ -1329,30 +1332,35 @@ def _create_datacfg_dict(cfg):
     datacfg.update({"MFScale": cfg["MFScale"]})
     datacfg.update({"DataTypeIDInFiles": cfg["DataTypeIDInFiles"]})
 
-    #s3 buckets
+    # s3 buckets
     if "bucket" in cfg:
         try:
             datacfg["s3_key"] = os.environ["S3_IN_KEY"]
             datacfg["s3_secret_key"] = os.environ["S3_IN_SECRET"]
         except KeyError:
             warn(
-                'Define environment variables S3_IN_KEY and S3_IN_SECRET'
-                ' to get input data from S3 buckets.')
+                "Define environment variables S3_IN_KEY and S3_IN_SECRET"
+                " to get input data from S3 buckets."
+            )
 
         if "s3path" in cfg:
             datacfg.update({"s3path": cfg["s3path"]})
         else:
-            warn('Unable to read data from s3 bucket. Define s3path')
+            warn("Unable to read data from s3 bucket. Define s3path")
         if "s3_url" in cfg:
             datacfg.update({"s3_url": cfg["s3_url"]})
         else:
-            warn('Unable to read data from s3 bucket. Define s3_url')
+            warn("Unable to read data from s3 bucket. Define s3_url")
 
         if "rm_s3_file" in cfg:
             datacfg.update({"rm_s3_file": cfg["rm_s3_file"]})
 
-        if ('s3path' in datacfg and 's3_url' in datacfg
-                and 's3_key' in datacfg and 's3_secret_key' in datacfg):
+        if (
+            "s3path" in datacfg
+            and "s3_url" in datacfg
+            and "s3_key" in datacfg
+            and "s3_secret_key" in datacfg
+        ):
             datacfg.update({"bucket": cfg["bucket"]})
 
     # Modify size of radar or radar spectra object
@@ -1791,14 +1799,14 @@ def _get_masterfile_list(datatypesdescr, starttimes, endtimes, datacfg, scan_lis
         )
         return [], None, None
 
-    if 'bucket' in datacfg:
+    if "bucket" in datacfg:
         masterfilelist = get_file_list_s3(
-            masterdatatypedescr, starttimes, endtimes, datacfg,
-            scan=masterscan)
+            masterdatatypedescr, starttimes, endtimes, datacfg, scan=masterscan
+        )
     else:
         masterfilelist = get_file_list(
-            masterdatatypedescr, starttimes, endtimes, datacfg,
-            scan=masterscan)
+            masterdatatypedescr, starttimes, endtimes, datacfg, scan=masterscan
+        )
 
     return masterfilelist, masterdatatypedescr, masterscan
 

src/pyrad_proc/pyrad/flow/flow_aux.py

@@ -97,7 +97,7 @@ def plot_timeseries(
     2018.03.05 -jgr- added x-limit of x axis to avoid unwanted error messages
 
     """
-    tvec = np.array(tvec) # convert  from pandas if needed
+    tvec = np.array(tvec)  # convert  from pandas if needed
     if period > 0:
         for i, data in enumerate(data_list):
             data *= period / 3600.0

src/pyrad_proc/pyrad/graph/plots_timeseries.py

@@ -527,7 +527,7 @@ def plot_ppi_map(
             )
             for rng_ring in rng_rings:
                 display_map.plot_range_ring(rng_ring, ax=ax)
-    
+
     if save_fig:
         for fname in fname_list:
             fig.savefig(fname, dpi=dpi)

src/pyrad_proc/pyrad/graph/plots_vol.py

@@ -253,7 +253,10 @@
 from .read_data_sensor import read_trt_thundertracking_traj_data  # noqa
 from .read_data_sensor import read_coord_sensors, read_disdro_parsivel  # noqa
 from .read_data_sensor import read_knmi  # noqa
-from .read_data_sensor import read_radiosounding_wyoming, read_radiosounding_igra #noqa
+from .read_data_sensor import (# noqa
+    read_radiosounding_wyoming,
+    read_radiosounding_igra,
+)
 from .read_data_sensor import read_fzl_igra  # noqa
 
 from .read_data_sun import read_sun_hits_multiple_days, read_sun_hits  # noqa

src/pyrad_proc/pyrad/io/__init__.py

@@ -1,15 +1,18 @@
 try:
     import fcntl
+
     FCNTL_AVAIL = True
 except ModuleNotFoundError:
     import msvcrt  # For windows
+
     FCNTL_AVAIL = False
 
 import os
 import time
 from warnings import warn
 import errno
 
+
 def lock_file(file):
     if FCNTL_AVAIL:
         while True:
@@ -47,8 +50,9 @@ def lock_file(file):
                 else:
                     raise
 
+
 def unlock_file(file):
-    if os.name == 'posix':
+    if os.name == "posix":
         fcntl.flock(file, fcntl.LOCK_UN)
     else:
-        msvcrt.locking(file.fileno(), msvcrt.LK_UNLCK, 1)
+        msvcrt.locking(file.fileno(), msvcrt.LK_UNLCK, 1)