Skip to content

portkey tracing #2044

Closed
spike-spiegel-21 wants to merge 8 commits intomainfrom
mayank/portkey_tracing
Closed

portkey tracing #2044
spike-spiegel-21 wants to merge 8 commits intomainfrom
mayank/portkey_tracing

Conversation

@spike-spiegel-21
Copy link
Copy Markdown
Contributor

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 13, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
deepeval Ready Ready Preview Comment Sep 18, 2025 1:48pm

Copy link
Copy Markdown
Contributor

@penguine-ip penguine-ip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are there any tool calling available as well in portkey?

@@ -0,0 +1,3 @@
from .patcher import instrument

__all__ = ["instrument"] No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to standardize the name

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will be using types for all the new integrations going forward. Please check the docs: confident-ai/confident-docs#11

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@spike-spiegel-21 is the idea that users will not be calling instrument? If that is the case i think we shouldn't include this as an export

@spike-spiegel-21
Copy link
Copy Markdown
Contributor Author

Are there any tool calling available as well in portkey?

We are logging the tool calls in the response in the LLM span.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants