122 factors best response aggregate recist rates

[AlexisCochard]

No description provided.

[DanChaltiel]

J'ai relu, c'est nickel 👍
Juste quelques commentaires mineurs.

Bien vu l'histoire de l'ordre des footnotes, il faudra peut-être brainstormer ça en réunion grstat (et on garde ça en tête pour #111, à voir si des lettres ne seraient pas plus lisibles que des astérisques)

Par contre je ne suis pas sûr de comprendre le problème initial (factor pour avoir tous les niveaux, dans l'ordre), même sans tes modifs le test passait bien :

grstat/tests/testthat/test-best_response.R

Line 98 in 3c20dda

test_that("No bug when no CR or PR", {

C'est quand on modifie best_response (genre as.character) avant de passer aggregate() ?

Si c'est ça, tu pourrais ajouter un test à cet endroit ?
Il faut un cas qui plantait avec ton code d'avant et que ta PR résout.