flotpython / course

Contenu pédagogique du MOOC Python3 sur fun-mooc.fr
Other
136 stars 22 forks source link

Roman class #40

Closed aollier closed 3 years ago

aollier commented 3 years ago

resolves #38

parmentelat commented 3 years ago

merci Adrien

à l'occasion on pourra refaire une passe pour ajouter quelques commentaires ;-)

aollier commented 3 years ago

Oui, quelques commentaires ne feront pas de mal. J'en profiterai pour remplacer

def __str__(self):
    return f"{self._roman}"

par

def __str__(self):
    return self._roman

vu que self._roman EST une str.

parmentelat commented 3 years ago

ok cool

et si tant qu'on y est on pouvait élargir le spectre des tests ce serait peut-être aussi bien :)

ps. à ce sujet il semble qu'il y a une régression récente quelque part dans la zone de ipywidgets en tous moi depuis quelque temps lorsque je fais exo.example() ou exo.correction(...) avec un exo de classe, le notebook cesse de fonctionner, il faut recharger la page (dans l'objet javascript notebook, plein de null sont ajoutés dans la liste des cellules, du coup tout est cassé...)

je suis le seul à avoir ça ou bien ?

aollier commented 3 years ago

En ce qui concerne le spectre des tests, si tu as des idées, je te propose d'en discuter par mail pour ne pas polluer ce fil de discussion. Et pour ce qui est de exo.example() et exo.correction() je n'ai pas de problème de mon côté. J'ai fait le test avec la classe Roman et tout va bien.

parmentelat commented 3 years ago

Et pour ce qui est de exo.example() et exo.correction() je n'ai pas de problème de mon côté. J'ai fait le test avec la classe Roman et tout va bien.

est-ce que tu pourrais me montrer la sortie de jupyter --version dans le contexte qui fonctionne pour toi ?

aollier commented 3 years ago

Capture d’écran 2020-11-22 122302

aollier commented 3 years ago

J'exécute jupyter dans le contexte du site web de https://fun-mooc.fr, je n'ai pas d'environnement à moi.