NVIDIA/Fuser

Reduce test verbosity.

Open

#3 020 ouverte le 25 sept. 2024

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (80 forks)github user discovery
Testinggood first issue

Métriques du dépôt

Stars
 (392 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

https://github.com/NVIDIA/Fuser/pull/2989 introduced "scheduleAndRun" utility function https://github.com/NVIDIA/Fuser/pull/2989/files#diff-fb98e47b0b389ef77407ef12badf08c42718a0e81dfce9ac5a6eca06feaca1c4R51. We could go one step further and include validation check in this function. e.g. this is the pattern that could go from two lines to one:

  auto cg_outputs =
      scheduleAndRun(&fusion, SchedulerType::PointWise, aten_inputs).outputs;
  testValidate(&fusion, cg_outputs, aten_inputs, __LINE__, __FILE__);

Guide contributeur

Direction de recherche
Trouvez la fonction scheduleAndRun dans le dépôt. Comprenez son implémentation actuelle et la fonction testValidate. Modifiez scheduleAndRun pour inclure un appel à testValidate avec fusion, outputs, inputs, numéro de ligne et nom de fichier. Ensuite, supprimez les appels de validation en double de toutes les utilisations.
Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
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.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75