bootcamp_python icon indicating copy to clipboard operation
bootcamp_python copied to clipboard

Python03 ex03

Open Mazoise opened this issue 3 years ago • 3 comments

  • Day: 03
  • Exercise: 03

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 :

def to_grayscale(self, array, filter, **kwargs):

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.

Mazoise avatar May 12 '22 12:05 Mazoise

Je pense que kwargs fonctionne aussi, je me cette issue en enhancment

qfeuilla avatar May 19 '22 17:05 qfeuilla

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

madvid avatar Aug 21 '22 18:08 madvid

J'ai fait une PR pour forcer l'utilisation de kwargs

madvid avatar Aug 21 '22 18:08 madvid