Skip to content
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

feat(groq): refactor groq, add groq tool call support #1133

Merged
merged 7 commits into from
Dec 20, 2024

Conversation

cjunkin
Copy link
Contributor

@cjunkin cjunkin commented Nov 23, 2024

resolves #1102

Implement tool call support and overhaul the Groq instrumentation following standard openinference implementation.

@cjunkin cjunkin requested a review from a team as a code owner November 23, 2024 01:12
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Nov 23, 2024
@cjunkin cjunkin self-assigned this Nov 26, 2024
@RogerHYang RogerHYang requested review from RogerHYang and removed request for a team December 2, 2024 17:15
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Dec 9, 2024
@cjunkin cjunkin changed the title fix(groq): display groq tool call details feat(groq): Groq tool call support Dec 9, 2024
@cjunkin cjunkin changed the title feat(groq): Groq tool call support feat(groq): groq tool call support Dec 9, 2024
@cjunkin cjunkin changed the title feat(groq): groq tool call support feat(groq): overhaul groq instrumentation and groq tool call support Dec 11, 2024
@cjunkin cjunkin changed the title feat(groq): overhaul groq instrumentation and groq tool call support feat(groq): overhaul groq instrumentation, groq tool call support Dec 11, 2024
cjunkin and others added 2 commits December 20, 2024 10:42
…s/test_tool_calls.py

Co-authored-by: Roger Yang <80478925+RogerHYang@users.noreply.github.com>
@cjunkin cjunkin changed the title feat(groq): overhaul groq instrumentation, groq tool call support feat(groq): refactor groq, add groq tool call support Dec 20, 2024
@cjunkin cjunkin merged commit 6057418 into Arize-ai:main Dec 20, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Bug] Groq Client Not Displaying Tool Choices in Tracing Dashboard
3 participants