Skip to content

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