flotpython / recreation

Zone récréative pour les étudiants du MOOC Python3
10 stars 7 forks source link

Natymundo #15

Closed natymundo closed 5 years ago

natymundo commented 5 years ago

J'y comprends pas grand chose, alors je copie mon dossier partout où je passe...

parmentelat commented 5 years ago

bonsoir

effectivement c'est un peu le bazar dans le repo maintenant :-)

manifestement vous avez utilisé plusieurs canaux différents pour mettre à jour le code, je vous joins une copie de l'état du repo, vous verrez qu'il y a deux branches distinctes sur lesquelles vous avez fait des ajouts aujourd'hui :)

bon ça n'est pas grave mais il va falloir que je fasse un coup de nettoyage, sans doute demain ...

Screenshot 2019-04-07 at 18 59 05
natymundo commented 5 years ago

oups... et avec tout ça je ne comprends toujours pas pourquoi je ne peux pas merger mes fichiers mis-à-jour sur la branche master (parce que finalement la branche natymundo ne sert à rien -enfin je crois) Existe-t-il un moyen simple de synchroniser par exemple le repo que j'ai sur mon profil avec le repo qui est ici? Est-ce à ça que sert le fork?? Bref, c'est encore assez flou et je m'excuse d'avance si le repo est encore pire demain...

parmentelat commented 5 years ago

ne vous excusez pas, ce n'est pas grave du tout, le repo est fait pour ça..
le truc c'est le cours est sur python et pas sur git, alors forcément :-)

donc oui la logique serait de merger les deux branches; sauf que dans l'état, j'ai bien regardé et à part le png je ne vois vraiment pas comment récupérer quoi que ce soit, car il y a vraiment eu des modifes pas franchement compatibles faites des deux cotés et je n'arrive pas à décider ce qu'il faut garder..

apparemment vous avez réussi à reconstruire dans master l'essentiel de ce qu'il y avait dans natymundo, donc le mieux c'est qu'on oublie cette branche là je pense

il faudra qu'un de ces jours j'écrive un mooc sur git donc :-)

bon courage; lorsque vous avez vaguement fini dites-moi et j'irai jeter un coup d'oeil à votre code

natymundo commented 5 years ago

Merci d'avoir pris le temps de me répondre!!

Oui, je crois que j'ai réussi à me dépatouiller. Du coup j'ai supprimé la branche natymundo et ferais mes modifs sur master. De toute façon il n'y a pas grand monde, donc je crois que ce n'est pas très grave.

Entre temps, je suis retournée voir le MOOC parce que je m'étais arrêtée en semaine 6 et je n'ai pas vu numpy... qui a l'air intéressant en effet. Trop puissant même à première vue -mais qui pourra peut-être résoudre la question subsidiaire de l'othello. A creuser!!

Mais je suis preneuse de toutes les remarques et suggestions, même sur mon quick-n-dirty-code!!

parmentelat commented 5 years ago

j'ai une première remarque qui est que les labels des boutons ne sont pas lisibles sur mon mac..
pas très grave car j'ai essayé plusieurs fois, mais je vais vite oublier :)

un petit coup de autopep8 serait bien aussi

je tacherai de regarder un de ces 4..

image

parmentelat commented 5 years ago

bonjour j'ai fait une petite passe rapide; voyez mes commentaires directement dans les sources en effet ce serait bien de faire une version avec numpy, et aussi de voir s'il y a lieu/moyen d'être un peu plus économe sur les allocations mémoire

sinon j'aime bien :)

je ne suis pas tout à fait sûr, mais je crois avoir rencontré un bug, voir screenshot, on ne me laisse pas jouer sur une case qui me semble pourtant OK mais bon je ne suis pas très expert en othello en fait :)

Screenshot 2019-04-10 at 14 18 48

natymundo commented 5 years ago

Grand merci pour ce retour!! (je n'ai pas encore regardé les commentaires) Je ne suis pas non plus experte en othello, et comme ce n'était pas clair dans les règles, j'ai considéré que 'à côté' ne comprenait pas les diagonales (bien que les jetons soient bien retournés dans les diagonales) Bon je cours voir les commentaires dans le code!!