[Model] Deprecate the score task (this will not affect users). #37537
[Model] Deprecate the score task (this will not affect users). #37537noooop merged 7 commits intovllm-project:mainfrom
Conversation
There was a problem hiding this comment.
Code Review
The pull request successfully deprecates the 'score' task, replacing it with 'classify' where appropriate and removing related logic and comments. The changes are consistent across tests, model configuration, entrypoints, and model executor components. The refactoring of activation functions and renaming of variables like pooled_data to embeddings or logits enhances code clarity and maintainability. No critical or high-severity issues were found in the changes.
|
Hi @noooop, the pre-commit checks have failed. Please run: uv pip install pre-commit>=4.5.1
pre-commit install
pre-commit run --all-filesThen, commit the changes and push to your branch. For future commits, Tip Is
|
|
Documentation preview: https://vllm--37537.org.readthedocs.build/en/37537/ |
vllm-project#37537)" This reverts commit ed359c4.
…project#37537) Signed-off-by: wang.yuqi <yuqi.wang@daocloud.io>
|
@noooop - this broke CI |
test below passed successfully when landing. let's rerun this test. All these tests passed. I also cannot reproduce this error locally. That's too strange. |


Purpose
Cross-encoder (aka reranker) models are a subset of classification models that accept two prompts as input and output num_labels equal to 1.
In the engine, we will no longer differentiate between classify and score. So we can deprecate the score task.
This is part of deprecate support for multi-task.
Test Plan
Test Result
Essential Elements of an Effective PR Description Checklist
supported_models.mdandexamplesfor a new model.