Open Mazoise opened 2 years ago
Je pense que kwargs fonctionne aussi, je me cette issue en enhancment
Il faut ça:
def to_grayscale(self, array, filter, *args):
mais le plus cohérent selon moi serait de changer le prototype à :
def to_grayscale(self, array, filter, weight=None): ou def to_grayscale(self, array, weight=None):
Mettre *args est utile si on ne connait pas le nombre d'argument qu'on va recevoir en paramètre, ce n'est pas le cas dans cet exo.
Alors c'est une vision de l'exercice que je trouve discutable, mais effectivement ça manque de cohérence
J'ai fait une PR pour forcer l'utilisation de kwargs
Le prototype de to_grayscale est pas bon, on devrait avoir *args plutôt que **kwargs en entrée vue qu'il y a pas de key sur l'argument weight
Examples If applicable, add examples to help explain your problem.
Au lieu de ça :
Il faut ça:
mais le plus cohérent selon moi serait de changer le prototype à :
Mettre *args est utile si on ne connait pas le nombre d'argument qu'on va recevoir en paramètre, ce n'est pas le cas dans cet exo.