flotpython / recreation

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

Par tranches #17

Closed natymundo closed 5 years ago

natymundo commented 5 years ago

J'ai revu le back de manière à gagner en efficacité. Je pourrais vérifier avec time, mais il me semble -avec ce que j'ai retenu des dictionnaires- que c'est plutôt efficace comme ça. Ceci dit, mon idée première avec mes 'tranches', était de faire des slices de booleens de ndarray que j'aurais pu 'coller' (ie multiplier terme à terme) facilement sur la table de jeu. Je ne maîtrise pas assez numpy pour ça, mais je crois que cette solution est assez belle. Ma première difficulté avec ndarray a été sa construction limitée à 32 termes !?? j'avoue ne pas avoir cherché beaucoup loin.

Pour la partie game, je ne sais pas comment modifier mes labels de tkinter pour qu'ils apparaissent sur votre PC. Je n'ai rien lu de tel dans la doc que j'ai en ma possession (de 2013...). Ca reste à creuser!

Merci pour l'éventuelle relecture.

parmentelat commented 5 years ago

bonjour

j'ai vu votre activité dans le repo; par contre comme je vois que vous ne maitrisez pas trop git - c'est tout à fait normal au début :) - je ne suis pas très sûr d'avoir bien compris quelle était la version à conserver ni comment interpréter les branches

en plus, j'ai vu que vous aviez ajouté dans le repo des fichiers en __pycache__, ce qui est sous-optimal, j'ai fait un petit nettoyage dans master (d'ailleurs vous n'êtes pas la seule à avoir ajouté ce genre de by-product)
mais je me dis à présent que je n'aurais peut-être pas dû car ça peut vous rendre la vie encore plus compliquée

je ne sais pas comment vous utilisez gihub; si vos voulez y comprendre quelque chose, moi perso je vous recommande d'utiliser un client lourd sur votre windows voici par exemple ce que me montre sourcetree actuellement

j'ai pris sur moi de merger Tableau-Brouillon dans master car sinon il y avait des scories de conflit qui trainaient dans master; mais si ce n'est pas ce qu'il fallait faire, dites-moi

en tous cas là à ce stade je préfère ne pas lire votre code avant que vous me disiez quelle est la bonne branche :)

image

natymundo commented 5 years ago

Bonjour.

Merci de prendre de votre dimanche pour réparer mes bêtises...

En fait, j'utilise GitHub DeskTop pour éditer en externe et lancer facilement le programme, ce qui me génère les pycache automatiquement dans le dossier courant.

J'interprète les branches comme des dossiers: nouvelle branche pour nouvelle construction du programme. Du coup, tableau-brouillon dans master correspond bien puisque c'est bien la même idée générale. Mais j'ai voulu créer une nouvelle branche parce que le tableau-brouillon n'est pas fonctionnel (alors que dans master, bien que pas optimisé, il avait le mérite de bien fonctionner). C'est là que tout s'est mélangé: j'ai essayé d'annuler des commits pour revenir à l'état précédent dans master, la nouvelle branche a copié master à un autre moment que celui que je croyais, j'ai voulu retourné à un autre état de fichier.... Bref! je crois que vous avez bien fait!

Ensuite, je ne sais pas à quel point mes mouvements vous sont notifiés. Je ne voudrais pas vous harceler! Mais github desktop met la pression pour pusher le moindre changement et pull-requester les nouvelles branches.

En tout cas, merci pour tout le temps que vous passez sur mon cas. Je vais essayer de me faire plus discrète à l'avenir et de vous présenter un code propre, c'est la moindre des choses.

parmentelat commented 5 years ago

On 14 Apr 2019, at 18:33, natymundo notifications@github.com wrote:

Bonjour.

Merci de prendre de votre dimanche pour réparer mes bêtises...

En fait, j'utilise GitHub DeskTop pour éditer en externe et lancer facilement le programme, ce qui me génère les pycache automatiquement dans le dossier courant.

J'interprète les branches comme des dossiers: nouvelle branche pour nouvelle construction du programme. Du coup, tableau-brouillon dans master correspond bien puisque c'est bien la même idée générale. Mais j'ai voulu créer une nouvelle branche parce que le tableau-brouillon n'est pas fonctionnel (alors que dans master, bien que pas optimisé, il avait le mérite de bien fonctionner). C'est là que tout s'est mélangé: j'ai essayé d'annuler des commits pour revenir à l'état précédent dans master, la nouvelle branche a copié master à un autre moment que celui que je croyais, j'ai voulu retourné à un autre état de fichier.... Bref! je crois que vous avez bien fait!

Ensuite, je ne sais pas à quel point mes mouvements vous sont notifiés. Je ne voudrais pas vous harceler! Mais github desktop met la pression pour pusher le moindre changement et pull-requester les nouvelles branches.

En tout cas, merci pour tout le temps que vous passez sur mon cas. Je vais essayer de me faire plus discrète à l'avenir et de vous présenter un code propre, c'est la moindre des choses.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

pas de souci, d’autant que c’est quand même moi qui vous envoie sur github sans aucune instruction préalable, c’est sûr que du coup quand on ne connait pas c’est space au début

natymundo commented 5 years ago

Bon, bah j'ai vérifié avec timeit et en fait ça n'est pas du tout efficace, mais alors vraiment pas du tout! Avec la toute première méthode j'étais à 16.57 (je n'ai pas l'unité) pour créer le jeu et faire jouer le joueur physique, avec la méthode 'du brouillon' je suis descendue à 0.94 (je n'ai toujours pas l'unité mais je suppose que c'est la même), et pour ma fabuleuse méthode 'par tranches', je n'ai jamais eu la réponse ne serait-ce de l'initialisation du jeu... Ca va sans doute plus vite après, mais si on ne peut pas initialiser... Donc j'abandonne cette branche (non sans une pointe de déception quand-même).

Du coup, je vous laisse regarder le code qui est dans master. Il reste des choses à faire pour avoir un programme complet, mais je pourrais au moins savoir si mon code est propre.

Merci de votre retour.