-
Notifications
You must be signed in to change notification settings - Fork 70
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
Add Portkey Observability Documentation #174
base: main
Are you sure you want to change the base?
Add Portkey Observability Documentation #174
Conversation
Hey @Bam4d can you take a look at this PR? |
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.
a few minor comments, but looks good to me outside of this. Thank you!
@@ -142,6 +142,68 @@ Here is a step-by-step [example](https://github.com/mistralai/cookbook/blob/main | |||
<img src="/img/guides/obs_langfuse2.png" alt="drawing" width="700"/> | |||
|
|||
|
|||
|
|||
Here's the PortkeyAI observability section in the same style as the Mistral docs: |
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.
thank you for the PR @siddharthsambharia-portkey
I am pretty sure you did not intend to add this ;) Can you remove it please?
Here's the PortkeyAI observability section in the same style as the Mistral docs: | ||
|
||
|
||
### Integration with PortkeyAI |
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.
Hey @siddharthsambharia-portkey thank you for this PR 🙏
May I ask you to add the block on PortkeyAI after the other observability tools already listed (that is, after phospho
?). I am sure you can see why we wouldn't want to go for a free for all here ;)
Thank you so very much!
print(response.choices[0].message) | ||
``` | ||
|
||
**Getting Started:** |
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.
would you add a link to the cookbook? Let's agree on which PR you rather have reviewed and then add it here! I am sure it would make using PortkeyAI muuuch easier :))
Closes and Fixes #175
This PR adds comprehensive documentation for integrating Portkey's observability features with Mistral AI models. Portkey provides unified observability across multiple LLM providers, offering cost tracking, latency monitoring, and request tracing capabilities.
Changes
Add new section Integration with PortkeyAI under the observability documentation