Skip to content

Par tranches#17

Closed
natymundo wants to merge 6 commits intomasterfrom
Par-tranches
Closed

Par tranches#17
natymundo wants to merge 6 commits intomasterfrom
Par-tranches

Conversation

@natymundo
Copy link
Collaborator

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.

Modifications de game.py pour coller avec les modifications de othello.py
Ajout d'une fin de jeu (à ameliorer)
ça n'a pas l'air d'aller plus vite, peut-être à cause du log.
J'avais fait la même erreur que dans l'essai précédent: je recréais 450 objets tranches à chaque tour de jeu.
Erreur réparéé: j'update mes tranches à chaque tour de jeu.
@parmentelat
Copy link
Contributor

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

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

parmentelat commented Apr 14, 2019 via email

@natymundo
Copy link
Collaborator Author

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.

@natymundo natymundo closed this Apr 16, 2019
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.

2 participants