Frédéric Chapoton
Frédéric Chapoton
the new module "planar_graphs" must also be listed in the meson.build of the `graph` folder
You also need to fix all the warnings in the "Lint" section of the continuous integration.
../src/sage/graphs/maps/meson.build:1:3: ERROR: File `__init__.py` does not exist. so remove the non-existing file in the list inside meson.build
Je me suis permis de faire un commit directement sur la branche pour montrer sur un fichier quelques problemes à corriger en général. donc, faire attention à bien faire `git...
Il faudrait corriger les problemes pénibles avec les "meson.build" et autres `__init__.py` ; je suggère de regarder comment ca a été fait dans #40023
le typage, ca me semble un peu voire très prematuré. C'est encore assez instable dans python. Et on n'a pas pris de decision globale dans sage. Je recommande d'eviter. Ca...
> Bonjour,merci pour votre commentaire. Nous pensions que le typage n'avait pas de conséquence tout en ajoutant du confort pour l'utilisateur, on s'était principalement inspiré de certaines méthodes dans sage...
les tests du nouveau repertoire passent tous pour moi voici une nouvelle suggestion: se debarasser des variables inutilisées, par exemple ``` pyflakes src/sage/graphs/maps/ src/sage/graphs/maps/rotating_permutation.py:152:9: local variable 'e' is assigned to...
@videlec : tu penses quoi de cette proposition sur les cartes planaires ?