feat(components): improve Baidu Wenxin chat model configuration#6140
Open
jimmyzhuu wants to merge 1 commit intoFlowiseAI:mainfrom
Open
feat(components): improve Baidu Wenxin chat model configuration#6140jimmyzhuu wants to merge 1 commit intoFlowiseAI:mainfrom
jimmyzhuu wants to merge 1 commit intoFlowiseAI:mainfrom
Conversation
Contributor
There was a problem hiding this comment.
Code Review
This pull request updates the ChatBaiduWenxin component to version 3.0, introducing dynamic model selection via asyncOptions and adding support for customModelName, topP, penaltyScore, and userId parameters. It also includes a new test suite and updates the global models configuration to include several ERNIE models. I have no feedback to provide.
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.
What changed
This PR improves the existing Baidu Wenxin chat model node by bringing it closer to the configuration experience of other chat model providers in Flowise.
Changes included:
Why this change is needed
Flowise already supports Baidu Wenxin as a chat provider, but the node was missing a few usability and parity features that exist in other chat model nodes:
This made the Baidu node less consistent and less convenient to use than other existing providers.
User impact
Users can now:
This keeps the node lightweight while making it much easier to use in real chatflow setups.
Tests / validation
Validated with:
Commands used:
Notes for reviewers
This PR intentionally keeps scope small: