Nicolas

Pour info je vais fusionner dans la branche principale la migration vers Python 3 et la lib requests. N'oubliez pas d'installer Python 3 : https://www.python.org/downloads/ Pour l'ajout des matchs de double il y a un truc emmerdant c'est que la page de simulation de classement ne les inclut pas. Je n'ai pas très envie de devoir faire 2 requêtes HTTP par joueur...

@Grumms est-ce qu'il y a eu une suite à ta discussion avec le gentil monsieur du comité 92 ? Et pas de réponse de la ligue ?

Pour info il y a eu récemment un sujet assez actif à propos du STB (et d'autres sujets plus ou moins liés comme l'organisation des tournois, l'échauffement, l'enchaînement des matchs) :

Oui, vu le faible nombre de points en jeu j'envisage de ne même pas aller chercher les palmarès des des adversaires de double ou alors de réduire la profondeur. Ou alors peut-être rendre cela possible mais optionnel.

Quand les pages de simulation fonctionneront de nouveau je regarderai ça. Mais si tu veux t'y coller et soumettre une pull request n'hésite pas

Ici il est question d'équité entre les 2 joueurs qui jouent un même match, pas entre tous ceux du tournoi. Enchaîner 2 matchs le même jour c'est faisable mais ça crée indéniablement une inéquité entre les joueurs. Personnellement je ne trouve pas ça scandaleux non plus, mais je trouve que les temps minimums de récupération du règlement pourraient être plus longs.

Depuis peu j'ai ajouté un avertissement dans l'outil quand l'utilisateur demande une profondeur de 3 ou plus, ça peut aider à limiter un peu les trop gros volumes de requêtes. C'est vrai que concernant le .exe il y a ce risque auquel je n'avais pas pensé. Je pense qu'il faut éviter de diffuser le fichier .exe seul, sans mettre au minimum un lien vers le github qui affiche le README ou vers ce thread.

Oui. D’ailleurs pour info je prévois de fusionner les versions Python 3 + requests dans la master d’ici peu vu que les retours ont été positifs (le bug que tu avais soulevé l’autre jour est corrigé).

Oui il y a 4 tentatives pour récupérer chaque palmarès. Si certains n’ont pas pu être récupérés après les 4 tentatives il y a un message qui est affiché pour le signaler. Je ferai quand même une correction pour mieux prendre en compte ce cas (c’est un problème spécifique à la version Requests).

C’est compris dedans.

J'ai fait un fichier .exe (encore merci @Paul91Essonne pour les infos). Il est disponible ici : https://github.com/coti/classement/releases/download/2019-08-25/palmares.exe C'est la version Requests + Python 3 avec toutes les corrections récentes.

https://github.com/coti/classement/archive/python3.zip Pour info ce lien c'est celui qu'on obtient quand on sélectionne la branche souhaitée puis qu'on clique sur le bouton vert ("Clone or download") en haut à droite de https://github.com/coti/classement

C’est bien le bug que tu as indiqué plus haut J’ai fait la correction sur la branche master et je l’ai incluse dans requests et python3 aussi.

PyInstaller produit un exécutable pour la plate-forme sur laquelle il est lancé. Donc pour faire un exe il faut le lancer sous Windows. J’ai justement vérifié ce point parce que je suis sur Mac.

@Yusuke118 merci pour les infos. Effectivement retirer les accents permet d'éliminer les erreurs d'encodage en pratique mais je pense que la meilleure solution est de passer à une gestion plus saine de unicode/bytes, ce que Python 2 permet très difficilement. Du coup j'en profite pour proposer cette version de test basée sur Python 3 : https://github.com/coti/classement/archive/python3.zip À part la migration Python 3 c'est la version requests. J'ai fait quelques changements dessus récemment. Voir ici les derniers commits : https://github.com/coti/classement/commits/python3 Merci à @ggman23 pour avoir signalé le bug sur la normalisation des classements "Top XX". Les changements de code pour Python 2 sont visibles ici : https://github.com/coti/classement/commit/1634ca7b757ca3cae0ccb2417fe9ecd77cede8da J'ai seulement un Mac mais je suis en train de m'installer une VM Windows. Je vais essayer de faire un exe avec.

Salut à tous Python 2.7 sera bientôt déclaré officiellement obsolète. Je commence donc à préparer le passage à Python 3. La première étape est de moderniser le code de connexion au site de la FFT (en particulier la gestion des cookies). Pour ce faire je l'ai basculé sur la librairie Requests, qui est la référence du domaine. C'est une dépendance externe, mais rassurez-vous : son installation est effectuée automatiquement si besoin au lancement de l'outil. Comme c'est un changement important je ne l'ai pas encore poussé sur la branche master. J'ai passé du temps à tester, mais pour l'instant je n'ai accès qu'à un Mac et je n'ai probablement pas pensé à tous les cas de figure. J'aurais donc besoin de quelques volontaires pour essayer cette nouvelle version avant de rendre le changement "officiel" en le poussant sur master. Vous pouvez la télécharger là : https://github.com/coti/classement/archive/requests.zip (Note : c'est toujours avec Python 2.7 qu'il faut lancer cette version ! Le passage à Python 3 sera pour plus tard) Merci de me faire des retours en MP pour me dire si ça fonctionne ou non chez vous

OK merci de l'avoir essayé. J'ai fait la correction : https://github.com/coti/classement/commit/35d118f475c84a4632a40a8450d636aaa5c32156 Je suis étonné que tu sois le premier à avoir ce problème. Tu utilises l'outil sous quel OS ? Et comment tu le lances ?

Est-ce que tu as des caractères spéciaux (comme des lettres avec des accents) dans ton nom d'utilisateur ou ton mot de passe ? J'ai la même erreur que toi quand je rentre un mot de passe incorrect. C'est possible que tu entres les bonnes valeurs mais qu'un bug d'encodage fasse que les mauvaises soient envoyées au serveur de la FFT.

Bizarre, ça fonctionne sans problème chez moi. La version 2.7.11 est assez ancienne maintenant. Est-ce que tu pourrais essayer avec la 2.7.16 ? Si ça ne fonctionne toujours pas je ferai une correction.

J'ai fait un changement pour éviter que l'outil reste bloqué dans cette situation : https://github.com/coti/classement/commit/2a7224ce062f2738db29f4dfd776a275cc207844 J'ai aussi ajouté un argument optionnel lors du lancement du script : -v pour afficher des logs de debug qui donnent (entre autres) des infos sur les requêtes au site de la FFT.

Oui, bonne remarque. C'est fait : https://github.com/coti/classement/commit/c5a3dccdd02e2a2f8d11b6c84c7054dd7e3b86d0

Encore une erreur d'encodage C'est corrigé : https://github.com/coti/classement/commit/6bcd6c5fdd52704148f4c2dc4fde4611dd09bc81

Nouvelle version pour corriger un bug remarqué grâce à @nico-lastar (merci ) : https://github.com/coti/classement/commit/e9243e946d1c5df0cd875083f0dee68a3c964201 C'est effectivement simple à implémenter. La seule difficulté est dans l'interface utilisateur. Je vais essayer de m'en occuper.

Nouveau changement pour réduire le volume d'informations affichée en sortie du programme : https://github.com/coti/classement/commit/ae1b2ed4adacd882f53b9889e9cf0d8af4878853 OK merci. Pour info j'ai corrigé le problème d'encodage que tu avais rapporté plus tôt. Mais avec le changement mentionné au-dessus ça ne devrait pas changer grand chose J'essaierai de faire aussi une distribution en .exe prochainement, en la mettant sur GitHub.

Je viens de pousser le changement pour la prise en compte des coefficients: https://github.com/coti/classement/commit/a7e0866e3d92c29c94e1de5766438f3c4794d6db