fix C416 unnecessary dict comprehension

Author: yannikschaelte

pyabc/distance/pnorm.py

@@ -252,11 +252,7 @@ def weights2dict(
         """
         # assumes that the summary statistic labels do not change over time
         return {
-            t: {
-                key: val
-                for key, val in zip(self.sumstat.get_ids(), weights[t])
-            }
-            for t in weights
+            t: dict(zip(self.sumstat.get_ids(), weights[t])) for t in weights
         }
 
 

pyabc/distance/util.py

@@ -60,7 +60,7 @@ def log_weights(
     log_file: File to log formatted output to.
     """
     # create weights dictionary with labels
-    weights = {key: val for key, val in zip(keys, weights[t])}
+    weights = dict(zip(keys, weights[t]))
 
     vals = [f"'{key}': {val:.4e}" for key, val in weights.items()]
     logger.debug(f"{label} weights[{t}] = {{{', '.join(vals)}}}")

pyabc/external/r/r_rpy2.py

@@ -30,7 +30,7 @@ def _dict_to_named_list(dct):
         or isinstance(dct, Parameter)
         or isinstance(dct, pd.core.series.Series)
     ):
-        dct = {key: val for key, val in dct.items()}
+        dct = dict(dct.items())
         # convert numbers, numpy arrays and pandas dataframes to builtin
         # types before conversion (see rpy2 #548)
         with conversion.localconverter(

pyabc/petab/amici.py

@@ -72,7 +72,7 @@ def __call__(self, par: Union[Sequence, Mapping]) -> Mapping:
 
         # convenience to allow calling model not only with dicts
         if not isinstance(par, Mapping):
-            par = {key: val for key, val in zip(self.x_free_ids, par)}
+            par = dict(zip(self.x_free_ids, par))
 
         # add fixed parameters
         for key, val in zip(self.x_fixed_ids, self.x_fixed_vals):