-
Notifications
You must be signed in to change notification settings - Fork 755
docs: add galileo observability example #1137
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
base: main
Are you sure you want to change the base?
Conversation
koverholt
left a comment
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.
Thanks for this PR and for your contribution! ❤️
I left a few formatting fixes as suggestions. Other than those, a big issue is that I tried the code example in this PR and I'm not seeing traces appear in Galileo.
It looks like the sample code in this PR uses GalileoSpanProcessor(), but the linked example in your sdk-examples repo uses a different approach with OTLPSpanExporter() + BatchSpanProcessor() and manually sets the endpoint and headers.
Could you update the sample code in this PR to match the working GitHub example, or a simplified version of it? Then we can re-test when ready. Thanks!
Co-authored-by: Kristopher Overholt <[email protected]>
Co-authored-by: Kristopher Overholt <[email protected]>
Co-authored-by: Kristopher Overholt <[email protected]>
Co-authored-by: Kristopher Overholt <[email protected]>
|
@koverholt Thank you so much for your review! I really appreciate it. |
|
Got it! Thanks for letting us know, now I see the open PR and understand the dependency & status. I'll label this as blocked for now while rungalileo/sdk-examples#144 is in progress. And let us know when we can re-test. Thanks! |
This PR adds Galileo, an AI evaluation and observability platform, to the docs.
Thank you!