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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants