Requested changes

AgentOps-AI · Jun 5, 2024 · 95c2cef · 95c2cef
1 parent 941be01
commit 95c2cef
Show file tree

Hide file tree

Showing 9 changed files with 372 additions and 331 deletions.
diff --git a/agentops/client.py b/agentops/client.py
@@ -262,8 +262,6 @@ def _record_event_sync(self, func, event_name, *args, **kwargs):
 
             event.returns = returns
 
-            # NOTE: Will likely remove in future since this is tightly coupled. Adding it to see how useful we find it for now
-            # TODO: check if screenshot is the url string we expect it to be? And not e.g. "True"
             if hasattr(returns, "screenshot"):
                 event.screenshot = returns.screenshot
 

diff --git a/agentops/helpers.py b/agentops/helpers.py
@@ -95,7 +95,7 @@ def default(o):
             return f"<<non-serializable: {type(o).__qualname__}>>"
 
     def remove_unwanted_items(value):
-        """Recursively remove unwanted items from dictionaries."""
+        """Recursively remove self key and None/... values from dictionaries so they aren't serialized"""
         if isinstance(value, dict):
             return {
                 k: remove_unwanted_items(v)

diff --git a/examples/multion/browse.ipynb b/examples/multion/browse.ipynb
diff --git a/examples/multion/retrieve.ipynb b/examples/multion/retrieve.ipynb
diff --git a/examples/multion/step.ipynb b/examples/multion/step.ipynb
diff --git a/examples/multion_example_browse.ipynb b/examples/multion_example_browse.ipynb
diff --git a/examples/multion_example_retrieve.ipynb b/examples/multion_example_retrieve.ipynb
diff --git a/examples/multion_example_step_session.ipynb b/examples/multion_example_step_session.ipynb
diff --git a/examples/multion_example_step_stream.ipynb b/examples/multion_example_step_stream.ipynb