Skip to content

Conversation

@dirkbrnd
Copy link
Contributor

@dirkbrnd dirkbrnd commented May 9, 2025

Summary

This adds Arize instrumentation via the OpenInference auto instrumentor. It requires the related PR to be merged first.

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Improvement
  • Model update
  • Other:

Checklist

  • Code complies with style guidelines
  • Ran format/validation scripts (./scripts/format.sh and ./scripts/validate.sh)
  • Self-review completed
  • Documentation updated (comments, docstrings)
  • Examples and guides: Relevant cookbook examples have been included or updated (if applicable)
  • Tested in clean environment
  • Tests added/updated (if applicable)

Additional Notes

Add any important context (deployment instructions, screenshots, security considerations, etc.)

@dirkbrnd dirkbrnd requested a review from a team as a code owner May 9, 2025 10:24
Copy link
Contributor

@manuhortet manuhortet left a comment

Choose a reason for hiding this comment

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

Great!!!

@dirkbrnd dirkbrnd changed the title Feat: Arize Phoenix via OpenInference feat: Arize Phoenix via OpenInference May 13, 2025
@dirkbrnd dirkbrnd merged commit 4b8c321 into main May 13, 2025
3 of 4 checks passed
@dirkbrnd dirkbrnd deleted the feat/arize-phoenix branch May 13, 2025 09:58
Mustafa-Esoofally pushed a commit that referenced this pull request Jun 4, 2025
## Summary

This adds Arize instrumentation via the OpenInference auto instrumentor.
It requires the [related
PR](Arize-ai/openinference#1603) to be merged
first.

## Type of change

- [ ] Bug fix
- [x] New feature
- [ ] Breaking change
- [ ] Improvement
- [ ] Model update
- [ ] Other:

---

## Checklist

- [ ] Code complies with style guidelines
- [ ] Ran format/validation scripts (`./scripts/format.sh` and
`./scripts/validate.sh`)
- [ ] Self-review completed
- [ ] Documentation updated (comments, docstrings)
- [ ] Examples and guides: Relevant cookbook examples have been included
or updated (if applicable)
- [ ] Tested in clean environment
- [ ] Tests added/updated (if applicable)

---

## Additional Notes

Add any important context (deployment instructions, screenshots,
security considerations, etc.)

---------

Co-authored-by: manuhortet <manuhortet@gmail.com>
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