Revert "(fix) standard logging metadata + add unit testing " #6381
+8
−98
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This reverts commit 8359cb6.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
krrishdholakia
pushed a commit
that referenced
this pull request
Oct 22, 2024
krrishdholakia
added a commit
that referenced
this pull request
Oct 23, 2024
* fix(utils.py): add 'disallowed_special' for token counting on .encode() Fixes error when '< endoftext >' in string * Revert "(fix) standard logging metadata + add unit testing (#6366)" (#6381) This reverts commit 8359cb6. * add new 35 mode lcard (#6378) * Add claude 3 5 sonnet 20241022 models for all provides (#6380) * Add Claude 3.5 v2 on Amazon Bedrock and Vertex AI. * added anthropic/claude-3-5-sonnet-20241022 * add new 35 mode lcard --------- Co-authored-by: Paul Gauthier <[email protected]> Co-authored-by: lowjiansheng <[email protected]> * test(skip-flaky-google-context-caching-test): google is not reliable. their sample code is also not working * Fix metadata being overwritten in speech() (#6295) * fix: adding missing redis cluster kwargs (#6318) Co-authored-by: Ali Arian <[email protected]> * Add support for `max_completion_tokens` in Azure OpenAI (#6376) Now that Azure supports `max_completion_tokens`, no need for special handling for this param and let it pass thru. More details: https://learn.microsoft.com/en-us/azure/ai-services/openai/concepts/models?tabs=python-secure#api-support * build(model_prices_and_context_window.json): add voyage-finance-2 pricing Closes #6371 * build(model_prices_and_context_window.json): fix llama3.1 pricing model name on map Closes #6310 * feat(realtime_streaming.py): just log specific events Closes #6267 * fix(utils.py): more robust checking if unmapped vertex anthropic model belongs to that family of models Fixes #6383 * Fix Ollama stream handling for tool calls with None content (#6155) * test(test_max_completions): update test now that azure supports 'max_completion_tokens' * fix(handler.py): fix linting error --------- Co-authored-by: Ishaan Jaff <[email protected]> Co-authored-by: Low Jian Sheng <[email protected]> Co-authored-by: David Manouchehri <[email protected]> Co-authored-by: Paul Gauthier <[email protected]> Co-authored-by: John HU <[email protected]> Co-authored-by: Ali Arian <[email protected]> Co-authored-by: Ali Arian <[email protected]> Co-authored-by: Anand Taralika <[email protected]> Co-authored-by: Nolan Tremelling <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reverting this just to eliminate what is causing issues on ci/cd
this PR had passed local testing