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

feat: inference anthropic extension #2885

Merged
merged 2 commits into from
May 11, 2024
Merged

feat: inference anthropic extension #2885

merged 2 commits into from
May 11, 2024

Conversation

henryh0x1
Copy link
Contributor

Describe Your Changes

  • Implement inference-anthropic-extension
  • Model list includes 3 most updated models including Claude 3 Opus, Claude 3 Sonnet, Claude 3 Haiku
  • Overwrite header, payLoad, and response based on anthropic API structure

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

@github-actions github-actions bot added the type: feature request A new feature label May 9, 2024
@henryh0x1 henryh0x1 requested review from a team and Van-QA May 9, 2024 09:58
@henryh0x1 henryh0x1 added the P2: nice to have Nice to have feature label May 9, 2024
namchuai
namchuai approved these changes May 11, 2024
View reviewed changes
Copy link
Contributor

@namchuai namchuai left a comment

Choose a reason for hiding this comment

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

LGTM

@Van-QA Van-QA merged commit efbc96d into dev May 11, 2024
7 checks passed
@Van-QA Van-QA deleted the feat/inference-anthropic-extension branch May 11, 2024 12:22
@github-actions github-actions bot added this to the v0.4.13 Broken Rice milestone May 11, 2024
@Van-QA Van-QA added the needs comms Needs to communicate changes/issue to users label May 14, 2024
@Van-QA
Copy link
Contributor

Van-QA commented May 31, 2024

#2777

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@namchuai namchuai namchuai approved these changes

@Van-QA Van-QA Awaiting requested review from Van-QA

Assignees

@henryh0x1 henryh0x1

Labels
needs comms Needs to communicate changes/issue to users P2: nice to have Nice to have feature type: feature request A new feature
Projects
Jan & Cortex
Archived in project
Milestone
v0.4.13 🌾 Broken Rice
Development

Successfully merging this pull request may close these issues.
3 participants
@henryh0x1 @Van-QA @namchuai