-
Notifications
You must be signed in to change notification settings - Fork 737
[Feature] support pool #3827
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
[Feature] support pool #3827
Changes from 16 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
8e0c8d4
support pool
lizexu123 57795ea
update pooling
lizexu123 302adb0
merge develop
lizexu123 87237b0
Merge branch 'develop' of https://github.com/PaddlePaddle/FastDeploy …
lizexu123 cef99ec
add pooler_config and check
lizexu123 926c796
update
lizexu123 a76a43e
support AutoWeightsLoader load weight
lizexu123 344a8df
fix
lizexu123 5ec6a93
update
lizexu123 2235785
merge develop
lizexu123 98b32fc
delete print
lizexu123 4f90dfc
Merge branch 'develop' of https://github.com/PaddlePaddle/FastDeploy …
lizexu123 0ef4c9a
update pre-commit
lizexu123 1daacb7
fix
lizexu123 9d1a011
fix xpu
lizexu123 2eff16e
fix ModelRegistry->model_registry
lizexu123 9fcd05e
fix Copilot review
lizexu123 945cba5
fix pooler.py
lizexu123 6f545aa
delete StepPooler
lizexu123 222d1b2
fix abstract
lizexu123 97b4649
fix default_loader_v1
lizexu123 4cf6164
fix Pre Commit
lizexu123 db0a4bf
support torch qwen3 dense
lizexu123 2681765
add test and fix torch-qwen
lizexu123 10969cd
fix
lizexu123 669d712
fix
lizexu123 dd45025
adapter ci:
lizexu123 31b7311
fix review
lizexu123 e023c6b
fix pooling_params.py
lizexu123 cb80ce8
fix
lizexu123 893cdbb
fix tasks.py 2025
lizexu123 d4dcc3c
fix print and logger
lizexu123 41aa2c5
Modefy ModelRegistry and delete AutoWeightsLoader
lizexu123 8e92eb4
fix logger
lizexu123 91f777e
fix test_embedding
lizexu123 a90a091
delete T
lizexu123 6f6c549
fix ci bug
lizexu123 1fdf477
ernie4_5 model_registry
lizexu123 9e4d1fa
fix test
lizexu123 ebf4e0c
fix test
lizexu123 27ec018
support Qwen3-Embedding-0.6B tp=1 load
lizexu123 adc5b8f
fix extra code
lizexu123 5e264f3
fix
lizexu123 798f788
delete fix vocab_size
lizexu123 b72ac60
delete prepare_params_dict
lizexu123 a1de646
fix:
lizexu123 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
Oops, something went wrong.
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.
改动ModelRegistry实现的位置和类名的原因是什么?这种不兼容的改动,非必要不更改
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.
这种兼容了旧的方式,vllm是放在了registry.py中,为了保持一致,类名并没有改,只是创建了一个实例
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.
调用位置改了,内部代码会跑不通
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.
这里之前ModelRegistry的调用我都做了相应的适配,ci也能过,应该不存在跑不通,内部代码是不开源的吗
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.
已经改回去了,在model_base.py中,兼容原本的方式,不用创建实例,可以直接用类方法