Skip to content
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

Roman class #40

Merged
merged 1 commit into from
Nov 21, 2020
Merged

Roman class #40

merged 1 commit into from
Nov 21, 2020

Conversation

aollier
Copy link
Contributor

@aollier aollier commented Nov 14, 2020

resolves #38

@aollier aollier force-pushed the roman branch 3 times, most recently from b18aa7a to a80ae83 Compare November 14, 2020 21:14
@parmentelat parmentelat merged commit 52c3c9a into flotpython:master Nov 21, 2020
@parmentelat
Copy link
Contributor

merci Adrien

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

@aollier aollier deleted the roman branch November 21, 2020 16:44
@aollier
Copy link
Contributor Author

aollier commented Nov 21, 2020

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
Copy link
Contributor

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
Copy link
Contributor Author

aollier commented Nov 21, 2020

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
Copy link
Contributor

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
Copy link
Contributor Author

aollier commented Nov 22, 2020

Capture d’écran 2020-11-22 122302

@aollier
Copy link
Contributor Author

aollier commented Nov 22, 2020

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

l'exo sur la classe Roman est cassé
2 participants