Skip to content

Conversation

abhishek9sharma
Copy link
Contributor

Add GUARDRAIL span kind to arize tracing so that kind reflects on arize frontend

PRimage

abhishek9sharma and others added 2 commits January 25, 2025 21:11
- Updated version to 0.6.3 in pyproject.toml.
- Add OpenInference span kind attribute to Guardrails telemetry spans.
@abhishek9sharma
Copy link
Contributor Author

Let me knnow if I need to change/fix something here

@abhishek9sharma
Copy link
Contributor Author

@CalebCourier @dtam any feedback on this

@CalebCourier
Copy link
Collaborator

Sorry @abhishek9sharma, we got caught up in some other projects. We'll take a look this week. Thanks for your patience!

@CalebCourier
Copy link
Collaborator

@abhishek9sharma I left a couple comments around treating openinference packages as optional dependencies. I'm going to go ahead and apply these changes so this PR can be included in today's release.

@CalebCourier
Copy link
Collaborator

Tests pass locally. They're failing here because of permissions issues since this is a fork.

Copy link
Collaborator

@CalebCourier CalebCourier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@abhishek9sharma I'm going to squash and merge this onto main to cleanup the commit history. Be sure to reset the HEAD of your fork to upstream/main afterwards.

@CalebCourier CalebCourier merged commit 51b4bc6 into guardrails-ai:main Feb 13, 2025
8 of 12 checks passed
@abhishek9sharma
Copy link
Contributor Author

@abhishek9sharma I left a couple comments around treating openinference packages as optional dependencies. I'm going to go ahead and apply these changes so this PR can be included in today's release.

Thanks @CalebCourier for this feedback and fixes. Apprectiate the merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants