huggingface/lighteval

[FT] Add tests for `VLLMModel` base methods

Ouverte

#724 ouverte le 15 mai 2025

Voir sur GitHub
 (0 commentaire) (1 réaction) (1 personne assignée)Python (514 forks)auto 404
featuregood first issue

Métriques du dépôt

Stars
 (2 496 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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 and also protect us a bit from silent, yet breaking changes on the vllm side

Solution/Feature

Add tests for VLLMModel

Guide contributeur

Direction de recherche
Regardez les tests existants dans le dépôt, en particulier pour les autres backends de modèles, pour comprendre les schémas de test. Inspectez ensuite les méthodes de la classe VLLMModel pour écrire des tests unitaires couvrant les cas limites et les échecs silencieux potentiels.
Stack technique
python
Domaine
testing
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Pythonpytest
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75