-
Notifications
You must be signed in to change notification settings - Fork 82
Span-level evals additional features #34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…tions, tools, etc. Tweak condense() logic to properly support structure.
…_type shows up in the final trace.
… Add type hinting for Tracer fields.
… the results after logged (important as it contains example_id).
…val API endpoint.
c44abe8
to
684b8ce
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Run UT's
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rerun UT's
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rerun UT's.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rerun UT's.
{"type": "output", "function": "test_func", "depth": 1, "timestamp": 1.2, "output": "result"}, | ||
{"type": "exit", "function": "test_func", "depth": 0, "timestamp": 2.0}, | ||
{"type": "enter", "function": "test_func", "depth": base_depth, "timestamp": 1.0}, | ||
{"type": "input", "function": "test_func", "depth": base_depth + 1, "timestamp": 1.1, "inputs": {"x": 1}}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch
mock_response.usage = MagicMock(prompt_tokens=10, completion_tokens=20, total_tokens=30) | ||
client.chat.completions.create = MagicMock(return_value=mock_response) | ||
mock_completion = MagicMock() | ||
mock_completion.choices = [MagicMock(message=MagicMock(content="test response"))] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
grape
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Nice PR -- I like the span names change. Seems like updating the UTs must've been annoying, thanks for doing this
Uh oh!
There was an error while loading. Please reload this page.