Skip to content

Conversation

@Anaisdg
Copy link

@Anaisdg Anaisdg commented Jan 13, 2026

This PR adds Galileo, an AI evaluation and observability platform, to the docs.

Thank you!

Copy link
Collaborator

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

Anaisdg and others added 4 commits January 14, 2026 15:02
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]>
@Anaisdg
Copy link
Author

Anaisdg commented Jan 14, 2026

@koverholt Thank you so much for your review! I really appreciate it.
Yes that is the new method that handles the headers automatically. Yes I've submitted a PR for the linked example. I'm working on getting the review there/for it to be merged. I'll update here when it has been merged. I apologize for jumping the gun there.

@koverholt
Copy link
Collaborator

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!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants