Feat: Add multi-provider LLM support (OpenAI + Gemini) - #43
Closed
dzakwanalifi wants to merge 1 commit into
Closed
dzakwanalifi wants to merge 1 commit into
dzakwanalifi wants to merge 1 commit into
Conversation
Add support for both OpenAI and Gemini providers with unified interface. Includes structured output support and improved error handling.
This was referenced Feb 9, 2026
|
Great work on this multi-provider abstraction! 🎉 This pattern would make it straightforward to add additional cloud providers. I've opened two related feature requests that could build on this PR:
Both would follow the same For Bedrock:
For SageMaker:
Would you be open to expanding this PR to include AWS providers, or would separate PRs be preferred? Happy to contribute the Bedrock/SageMaker implementations if helpful. |
Collaborator
|
Thanks @dzakwanalifi! Multi-provider support (Gemini included) now ships via LiteLLM (#168, merged) — Gemini works with a |
GhislainAdon
pushed a commit
to GhislainAdon/iroko-rag
that referenced
this pull request
Jul 6, 2026
Read PIFS embedding model, dimensions, provider, and timeout from the persisted PIFS config when constructing CLI workspaces. Also print ask final output when streaming tool events.
This file contains hidden or 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
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.
Problem
PageIndex currently only supports OpenAI models, limiting user choice and potentially increasing costs.
Solution
Add unified interface supporting both OpenAI GPT-4 and Gemini 2.5 Flash models with structured output capabilities.
Key Changes
utils.pyconfig.yamlpage_index.pyCode Changes
utils.py: AddedLLMProviderclass with unified interfaceconfig.yaml: Added provider configuration optionpage_index.py: Enhanced error handling and Gemini integrationUse Case
Users can now choose between providers:
This provides flexibility for different cost/performance requirements while maintaining full compatibility.