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

fix(agents-api): Fix embedding model litellm error #870

Merged
merged 2 commits into from
Nov 22, 2024

Conversation

HamadaSalhab
Copy link
Contributor

@HamadaSalhab HamadaSalhab commented Nov 22, 2024

Important

Commented out a line in aembedding() in litellm.py to prevent errors with voyage/voyage-3 model.

  • Behavior:
    • Commented out line in aembedding() in litellm.py to prevent errors with voyage/voyage-3 model.
    • The line model = f"openai/{model}" is temporarily disabled.
  • Misc:
    • No other changes or fixes are included in this PR.

This description was created by Ellipsis for 3d429be. It will automatically update as commits are pushed.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 3d429be in 8 seconds

More details
  • Looked at 16 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. agents-api/agents_api/clients/litellm.py:61
  • Draft comment:
    Consider adding a comment explaining why the code is commented out and any conditions under which it should be re-enabled. This will help future developers understand the context and necessity of this change.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The commented-out code is a temporary fix, but it should be clearly marked for future developers to understand the context and necessity of the change.

Workflow ID: wflow_pMIWvLZ8808EZy8g


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

Copy link
Contributor

@standard-input standard-input bot left a comment

Choose a reason for hiding this comment

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

No issues flagged.
Standard Input can make mistakes. Check important info.

@HamadaSalhab HamadaSalhab merged commit 5b63886 into dev Nov 22, 2024
4 checks passed
@HamadaSalhab HamadaSalhab deleted the x/recall-default-false branch November 22, 2024 09:00
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.

1 participant