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.

Sign up for GitHub

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

Jump to bottom

Add Tags for LLMs #6229

Merged
merged 2 commits into from
Jun 15, 2023
Merged

Add Tags for LLMs #6229

merged 2 commits into from
Jun 15, 2023

Conversation

vowelparrot
Copy link
Contributor

  • Add tracing tags to LLMs + Chat Models (both inheritable and local)
  • Add tags for the run_on_dataset helper function(s)

@vowelparrot
Add Tags for LLMs
56d246d 
- [x] Add tracing tags to LLMs + Chat  Models (both inheritable and local)
- [x] Add tags for the run_on_dataset helper function(s)
agola11
agola11 approved these changes Jun 15, 2023
View reviewed changes
Copy link
Collaborator

@agola11 agola11 left a comment

Choose a reason for hiding this comment

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

lgtm

@vowelparrot vowelparrot merged commit ae76e47 into master Jun 15, 2023
@vowelparrot vowelparrot deleted the vwp/tag_llms_and_helprs branch June 15, 2023 18:24
This was referenced Jun 25, 2023
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@agola11 agola11 agola11 approved these changes

@nfcampos nfcampos Awaiting requested review from nfcampos

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vowelparrot @agola11