θΏ™ζ˜―indexlocζδΎ›ηš„ζœεŠ‘οΌŒδΈθ¦θΎ“ε…₯任何密码
Skip to content

Conversation

@suzuki-shm
Copy link
Contributor

@suzuki-shm suzuki-shm commented Apr 21, 2025

Pull Request Type

  • ✨ feat
  • πŸ› fix
  • ♻️ refactor
  • πŸ’„ style
  • πŸ”¨ chore
  • πŸ“ docs

Relevant Issues

resolves #3689

What is in this change?

  • Updated the implementation to use the openai library instead of @azure/openai
    • Since the Embeddings API requires an API version to be specified, set it to 2024-12-01-preview to match the ChatCompletion API
  • Removed @azure/openai
  • Upgraded the openai library to the latest version
  • Since gpt-3.5-turbo, which is used by default in OPEN_MODEL_PREF, has already been deprecated, I replaced it with gpt-4o-mini, a lightweight model.

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
    • Checked talking with LLM on AOAI
    • Test using Embedding Engine on AOAI
  • Docker build succeeds locally

@timothycarambat timothycarambat merged commit cd900f9 into Mintplex-Labs:master Apr 29, 2025
cabwds pushed a commit to cabwds/anything-llm that referenced this pull request Jul 3, 2025
Mintplex-Labs#3691)

* Replace @azure/openai to OpenAI lib

* Remove @azure/openai dependency and update openai to version 4.95.1

* linting

* update logging
fix translation dictionary error

* remove bad ENV key that DNE
linting
Patch Azure OpenAI
Migrate Azure Agent provider to use OpenAI Schema for tool calling performance

* unset

* migrate azure to use default OAI stream handler

---------

Co-authored-by: Timothy Carambat <rambat1010@gmail.com>
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.

[FEAT]: Replace @azure/openai with openai

2 participants