huggingface / huggingface/lighteval
[FT] Add tests for `VLLMModel` base methods
Open
feature
good first issue
- Dominant language
- Python
- Stars
- 2.5k
- Forks
- 555
- Avg merge
- 1d 6h
- Merged PRs (30d)
- 1
Description
## Issue encountered
From @lewtun: in addition to adding integration tests for the vllm backend of lighteval WDYT about prioritising some unit tests for the VLLMModel class methods? That would help catch subtle bugs like [this](https://github.com/huggingface/lighteval/pull/721) and also protect us a bit from silent, yet breaking changes on the vllm side
## Solution/Feature
Add tests for `VLLMModel`
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.