Skip to content

Chat LLM call refactor#111

Merged
zmccormick7 merged 10 commits into
mainfrom
gemini_2_5_chat_support
May 12, 2025
Merged

Chat LLM call refactor#111
zmccormick7 merged 10 commits into
mainfrom
gemini_2_5_chat_support

Conversation

@zmccormick7

Copy link
Copy Markdown
Collaborator

Chat LLM call updates

  • upgrades Gemini SDK to newer google-genai version
  • moves get_response code to chat folder, since that's the only place it's used
  • changes model name specification to {provider}/{model_name} (old method remains for backwards compatibility)
  • removes unused non-Instructor handler functions
  • adds special handling needed for OpenAI o-series models

Unrelated

  • removes Milvus integration test because the MilvusDB class requires special configuration to run via GitHub Actions

@zmccormick7 zmccormick7 requested a review from NickGator1 May 5, 2025 22:06
NickGator1
NickGator1 previously approved these changes May 5, 2025
NickGator1
NickGator1 previously approved these changes May 6, 2025
NickGator1
NickGator1 previously approved these changes May 9, 2025
@zmccormick7 zmccormick7 merged commit f56f7a2 into main May 12, 2025
1 check failed
@zmccormick7 zmccormick7 deleted the gemini_2_5_chat_support branch May 12, 2025 18:35
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.

2 participants