forked from router-for-me/CLIProxyAPI
-
-
Notifications
You must be signed in to change notification settings - Fork 40
v6.6.71 #75
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
Merged
Merged
v6.6.71 #75
Changes from 12 commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
08ab6a7
feat(gemini): add per-key model alias support for Gemini provider
hkfires 70fdd70
refactor(cliproxy): extract generic buildConfigModels function for mo…
hkfires ce7474d
feat(cliproxy): propagate thinking support metadata to aliased models
hkfires 857c880
Merge pull request #785 from router-for-me/gemini
hkfires b055e00
fix(executor): use upstream model for thinking config and payload tra…
hkfires 96340bf
refactor(executor): resolve upstream model at conductor level before …
hkfires 26efbed
refactor(executor): remove redundant upstream model parameter from tr…
hkfires c6b0e85
Fixed: #790
luispater e947266
Merge pull request #795 from router-for-me/modelmappings
hkfires 2c01b2e
feat(watcher): add Gemini models and OAuth model mappings change dete…
hkfires e0381a6
refactor(watcher): extract model summary functions to dedicated file
hkfires cb56cb2
Merge pull request #800 from router-for-me/modelmappings
hkfires e85c9d9
Merge branch 'main' into plus
luispater File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The current implementation of
LookupStaticModelInfore-creates a slice of all model groups on every call. For better performance and cleaner code, consider initializing a map of all static models once and then performing a simple map lookup. This would be more efficient, especially if the number of models grows.You can add the following at the package level:
And then replace the function with this suggestion.