Support Bailing LLM from ALIPAY for #3487 #3543
Open
Conversation
This file contains 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
cuauty
changed the title
|
Please take a look at this pull request. If you have any questions, please feel free to contact us. Thank you very much. |
cuauty
changed the title
|
Could you help review this PR for Bailing LLM from ALIPAY? If any comments please let us know. Thank you. |
|
I just change the file "fastchat/serve/gradio_web_server.py" in this PR and use new Context class replacing List. This change can make the command 'python -m fastchat.serve.gradio_web_server --controller "" --share --register xxxx' cowork with your code sync in PR #3546. Could you help review this PR? If any comments please let me know. |
cuauty
force-pushed
the
bailing_api_fs
branch
from
39a9d30 to
c8dda13
Compare
|
LGTM |
cuauty
force-pushed
the
bailing_api_fs
branch
from
c8dda13 to
ff56f18
Compare
cuauty
force-pushed
the
bailing_api_fs
branch
from
ff56f18 to
ba8a246
Compare
cuauty
force-pushed
the
bailing_api_fs
branch
from
ba8a246 to
e5da119
Compare
leywar
reviewed
Dec 11, 2024
Define the conversation template for Bailing LLM and provide the iterator.
Including: temperature top_p top_k max_tokens
New code defines the Context class and use it as input parameter.
cuauty
force-pushed
the
bailing_api_fs
branch
from
c9a9a46 to
660ea0a
Compare
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.
Why are these changes needed?
This PR implements the support for Bailing reasoning. The Bailing LLM provides the HTTP end point for reasoning and can be acessed by HTTP Post.
Related issue number (if applicable)
Close #3487
Checks
format.shto lint the changes in this PR.