Use list literals

Author: Armavica

pymc/variational/opvi.py

@@ -526,11 +526,11 @@ def collect_shared_to_list(params):
     List
     """
     if isinstance(params, dict):
-        return list(
+        return [
             t[1]
             for t in sorted(params.items(), key=lambda t: t[0])
             if isinstance(t[1], pytensor.compile.SharedVariable)
-        )
+        ]
     elif params is None:
         return []
     else:

tests/logprob/test_utils.py

@@ -151,8 +151,7 @@ def test_intermediate_rv(self):
         res_ancestors = list(ancestors((res,)))
 
         assert (
-            len(list(n for n in res_ancestors if n.owner and isinstance(n.owner.op, MeasurableOp)))
-            == 1
+            len([n for n in res_ancestors if n.owner and isinstance(n.owner.op, MeasurableOp)]) == 1
         )
 
         assert c_value_var in res_ancestors