pyglove 0.4.5.dev202410210809__tar.gz → 0.4.5.dev202410250809__tar.gz

{pyglove-0.4.5.dev202410210809 → pyglove-0.4.5.dev202410250809}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: pyglove
-Version: 0.4.5.dev202410210809
+Version: 0.4.5.dev202410250809
 Summary: PyGlove: A library for manipulating Python objects.
 Home-page: https://github.com/google/pyglove
 Author: PyGlove Authors

{pyglove-0.4.5.dev202410210809 → pyglove-0.4.5.dev202410250809}/pyglove/core/object_utils/profiling.py

@@ -41,6 +41,40 @@ class TimeIt:
       """Returns whether the context has error."""
       return self.error is not None
 
+  @dataclasses.dataclass
+  class StatusSummary:
+    """Aggregated summary for repeated calls for `pg.timeit`."""
+
+    @dataclasses.dataclass
+    class Entry:
+      """Aggregated status from the `pg.timeit` calls of the same name."""
+
+      num_started: int = 0
+      num_ended: int = 0
+      num_failed: int = 0
+      avg_duration: float = 0.0
+
+      def update(self, status: 'TimeIt.Status'):
+        self.avg_duration = (
+            (self.avg_duration * self.num_started + status.elapse)
+            / (self.num_started + 1)
+        )
+        self.num_started += 1
+        if status.has_ended:
+          self.num_ended += 1
+        if status.has_error:
+          self.num_failed += 1
+
+    breakdown: dict[str, 'TimeIt.StatusSummary.Entry'] = (
+        dataclasses.field(default_factory=dict)
+    )
+
+    def aggregate(self, timeit_obj: 'TimeIt'):
+      for k, v in timeit_obj.status().items():
+        if k not in self.breakdown:
+          self.breakdown[k] = TimeIt.StatusSummary.Entry()
+        self.breakdown[k].update(v)
+
   def __init__(self, name: str):
     self._name = name
     self._start_time = None

{pyglove-0.4.5.dev202410210809 → pyglove-0.4.5.dev202410250809}/pyglove/core/object_utils/profiling_test.py

@@ -101,6 +101,38 @@ class TimeItTest(unittest.TestCase):
     self.assertTrue(r['node.child.grandchild'].has_error)
     self.assertTrue(t2.has_error)
 
+  def test_timeit_summary(self):
+    summary = profiling.TimeIt.StatusSummary()
+    for i in range(10):
+      with profiling.timeit('node') as t:
+        time.sleep(0.1)
+        with profiling.timeit('child'):
+          time.sleep(0.1)
+          try:
+            with profiling.timeit('grandchild'):
+              time.sleep(0.1)
+              if i < 2:
+                raise ValueError('error')
+          except ValueError:
+            pass
+      summary.aggregate(t)
+    self.assertEqual(
+        list(summary.breakdown.keys()),
+        ['node', 'node.child', 'node.child.grandchild']
+    )
+    self.assertEqual(
+        [x.num_started for x in summary.breakdown.values()],
+        [10, 10, 10]
+    )
+    self.assertEqual(
+        [x.num_ended for x in summary.breakdown.values()],
+        [10, 10, 10]
+    )
+    self.assertEqual(
+        [x.num_failed for x in summary.breakdown.values()],
+        [0, 0, 2]
+    )
+
 
 if __name__ == '__main__':
   unittest.main()

{pyglove-0.4.5.dev202410210809 → pyglove-0.4.5.dev202410250809}/pyglove/core/symbolic/functor.py

@@ -375,7 +375,7 @@ class Functor(pg_object.Object, object_utils.Functor):
     if (
         signature.return_value
         and flags.is_type_check_enabled()
-        and return_value != pg_typing.MISSING_VALUE
+        and pg_typing.MISSING_VALUE != return_value
     ):
       return_value = signature.return_value.apply(
           return_value, root_path=self.sym_path + 'returns'

{pyglove-0.4.5.dev202410210809 → pyglove-0.4.5.dev202410250809}/pyglove.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: pyglove
-Version: 0.4.5.dev202410210809
+Version: 0.4.5.dev202410250809
 Summary: PyGlove: A library for manipulating Python objects.
 Home-page: https://github.com/google/pyglove
 Author: PyGlove Authors