Skip to content

Add Portkey Observability Documentation #174

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

siddharthsambharia-portkey
Copy link

@siddharthsambharia-portkey siddharthsambharia-portkey commented Jan 31, 2025

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

  • Include Python code examples for integration
  • Add descriptions of key observability features
  • Add links to PortkeyAI documentation and GitHub repository

@siddharthsambharia-portkey
Copy link
Author

Hey @Bam4d can you take a look at this PR?

Copy link
Collaborator

@fracapuano fracapuano left a 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!

Here's the PortkeyAI observability section in the same style as the Mistral docs:


### Integration with PortkeyAI
Copy link
Collaborator

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!

@fracapuano
Copy link
Collaborator

Hey @siddharthsambharia-portkey 👋 Just circling back on this PR to see if you'd be still interested in contributing portkey.

Thank you!

siddharthsambharia-portkey added 2 commits March 10, 2025 11:56
@siddharthsambharia-portkey
Copy link
Author

Hey @fracapuano i have made the suggested changes, LGTM?

@fracapuano
Copy link
Collaborator

Hey @siddharthsambharia-portkey, looks like conflicts prevent merging. Can you fix those? happy to review after that! 😊

@siddharthsambharia-portkey
Copy link
Author

@fracapuano done! can you take a look?

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.

Add Portkey AI Observability Docs
2 participants