-
Notifications
You must be signed in to change notification settings - Fork 127
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Python03 ex03 #240
Comments
Je pense que kwargs fonctionne aussi, je me cette issue en enhancment |
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.
The text was updated successfully, but these errors were encountered: