Incompatibilité PyReverse <-> Random ?

Version imprimable

Bonjour à tous

Alors voici un problème sur lequel je me casse la tête depuis le début de l'après midi. Je souhaite faire un diagramme de classe de mon code. J'utilise pour cela l'outil Pyreverse (inclut dans pylint, j'ai la version 2.4.4).

La commande que j'utilise pour générer un fichier classes.gv (dont je me dépatouille après) est :

Code:

Pyreverse -o gv -A -my -S test.py

Jusque là tout fonctionnait bien. Jusqu'à ce que je refasse tourner la chose tout à l'heure, et que j'obtienne une erreur :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 parsing test.py... Traceback (most recent call last): File "C:\ProgramData\Anaconda3\Scripts\pyreverse-script.py", line 10, in <module> sys.exit(run_pyreverse()) File "C:\ProgramData\Anaconda3\lib\site-packages\pylint\__init__.py", line 37, in run_pyreverse PyreverseRun(sys.argv[1:]) File "C:\ProgramData\Anaconda3\lib\site-packages\pylint\pyreverse\main.py", line 184, in __init__ sys.exit(self.run(args)) File "C:\ProgramData\Anaconda3\lib\site-packages\pylint\pyreverse\main.py", line 209, in run writer.DotWriter(self.config).write(diadefs) File "C:\ProgramData\Anaconda3\lib\site-packages\pylint\pyreverse\writer.py", line 38, in write self.write_classes(diagram) File "C:\ProgramData\Anaconda3\lib\site-packages\pylint\pyreverse\writer.py", line 59, in write_classes self.printer.emit_node(i, **self.get_values(obj)) File "C:\ProgramData\Anaconda3\lib\site-packages\pylint\pyreverse\writer.py", line 134, in get_values args = [arg.name for arg in func.args.args if arg.name != "self"] TypeError: 'NoneType' object is not iterable
J'ai réussi à trouver un exemple minimal, à coup de commentage de gros bloc de code par dichotomie. Voici un code que Pyreverse digère bien :
Code:

1 2 3 4 5 import random class randompick(): def __init__(self): self.rrand = None
et un autre qui produit l'erreur :
Code:

1 2 3 4 5 import random class randompick(): def __init__(self): self.rrand = random.Random()
Donc manifestement il y a qqch qui se passe pas bien avec la lib random. Mais alors quoi ?
Que puis je faire de plus de mon côté ? Je viens de détecter un bug pylint non ? Car avant ce truc fonctionnait, et j'ai du mettre à jour entre temps...

04/05/2020, 19h40
wiztricks

Salut,

Citation:

Envoyé par lg_53

Car avant ce truc fonctionnait, et j'ai du mettre à jour entre temps...

Allez voir si le problème est déjà répertorié dans la buglist.
Prenez le temps de vérifier que c'est bien la nouvelle version qui...
Ouvrez un bug-report.

- W
05/05/2020, 14h50
lg_53

Merci pour ce retour wiztricks. Bon bah c'est parti pour la rédaction de mon premier report bug alors !

https://github.com/PyCQA/pylint/issues/3589

On verra bien ce que ça donne !
05/05/2020, 15h17
wiztricks

Citation:

Envoyé par lg_53

Bon bah c'est parti pour la rédaction de mon premier report bug alors !

çà se fête... :lahola:

- W

Bonjour

Citation:

Envoyé par lg_53

et un autre qui produit l'erreur :

Code:

1
2
3
4
5
import random
 
class randompick():    
    def __init__(self):
        self.rrand = random.Random()

Donc manifestement il y a qqch qui se passe pas bien avec la lib random. Mais alors quoi ?
Que puis je faire de plus de mon côté ? Je viens de détecter un bug pylint non ? Car avant ce truc fonctionnait, et j'ai du mettre à jour entre temps...

Perso chez-moi pyreverse analyse ce code très correctement (pylint 1.9.4, debian Buster)

Le résultat

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
digraph classes {
	graph [bb="0,0,275.5,567",
		charset="utf-8",
		rankdir=BT
	];
	node [label="\N"];
	0	 [height=2.0278,
		label="{_random.Random|\l|getrandbits()\lgetstate()\ljumpahead()\lrandom()\lseed()\lsetstate()\l}",
		pos="71,494",
		rects="0,543.5,142,566.5 0,519.5,142,543.5 0,421.5,142,519.5",
		shape=record,
		width=1.9722];
	1	 [height=0.97222,
		label="{a.randompick|rrand\l|}",
		pos="218,494",
		rects="160.5,505.5,275.5,528.5 160.5,482.5,275.5,505.5 160.5,459.5,275.5,482.5",
		shape=record,
		width=1.5972];
	2	 [height=5.1389,
		label="{random.Random|VERSION : int\lgauss_next : NoneType\l|betavariate()\lchoice()\lexpovariate()\lgammavariate()\lgauss()\lgetstate()\ljumpahead()\llognormvariate()\lnormalvariate()\lparetovariate()\lrandint()\lrandrange()\lsample()\lseed()\lsetstate()\lshuffle()\ltriangular()\luniform()\lvonmisesvariate()\lweibullvariate()\l}",
		pos="144,185",
		rects="52,346.5,236,369.5 52,308.5,236,346.5 52,0.5,236,308.5",
		shape=record,
		width=2.5556];
	2 -> 0	 [arrowhead=empty,
		arrowtail=none,
		pos="e,88.157,421.38 100.38,369.65 96.955,384.14 93.621,398.25 90.5,411.46"];
	2 -> 1	 [arrowhead=diamond,
		arrowtail=none,
		fontcolor=green,
		label=rrand,
		lp="216,395.5",
		pos="e,209.73,459.47 188.22,369.65 195.14,398.56 201.71,425.99 206.91,447.7",
		style=solid];
}

05/05/2020, 18h36
wiztricks

Citation:

Envoyé par Sve@r

Perso chez-moi pyreverse analyse ce code très correctement (pyint 1.9.4, debian Buster)

Un peu de lecture du post original indique qu'il faut avoir installé au moins 2.4.4 pour gagner le gros lot... et qu'il est apparu après une mise à jour de...

- W
05/05/2020, 20h54
Sve@r

1 pièce(s) jointe(s)

Citation:

Envoyé par wiztricks

Un peu de lecture du post original indique qu'il faut avoir installé au moins 2.4.4 pour gagner le gros lot... et qu'il est apparu après une mise à jour de...

Ben voilà. Ca marche aussi avec une version de "au-moins" 2.4.4 que j'ai installée tout exprès pour toi.
Pièce jointe 563231
05/05/2020, 22h29
wiztricks

Citation:

Envoyé par Sve@r

Ben voilà. Ca marche aussi avec une version de "au-moins" 2.4.4 que j'ai installée tout exprès pour toi.

Reste plus qu'à comprendre/faire marcher çà sur la machine de lg_53... qui tourne avec une autre version de Python, sur un autre OS....

Dit autrement, tant qu'on ne sait pas reproduire le problème et qu'il y une différence, on recommence.

- W
05/05/2020, 23h13
lg_53

Est ce que vous pouvez tester avec la version 2.4.4 précisément ?

Car il me semble, dans mes souvenirs du temps où ça marchait, j'étais en 1.9.qqch, donc que vous me disiez que dans cette version là ca fonctionne ca ne m'étonne pas.
05/05/2020, 23h43
Sve@r

Citation:

Envoyé par wiztricks

Dit autrement, tant qu'on ne sait pas reproduire le problème et qu'il y une différence, on recommence.

Dit autrement, peut-être que le souci ne vient pas que de pylint tout seul.

Citation:

Envoyé par lg_53

Est ce que vous pouvez tester avec la version 2.4.4 précisément ?

J'y ai pensé aussi. Malheureusement j'ai essayé de trouver la version 2.4.4 sur le site de pylint sans succès. La version que j'ai installée l'a été avec la commande qu'ils indiquent sur leur site git clone https://github.com/PyCQA/pylint mais ça télécharge juste un clone de la dernière version (2.6.0) déposée sur github. J'aurais pensé trouver sur leur site des archives de version comme on en trouve sur d'autres outils (VirtuaBox, Postgres, etc).
Maintenant si on pensait dans l'autre sens en se disant que si la dernière version de leur site (2.6.0) fonctionne aussi chez-toi, cela pourrait définitivement faire disparaitre ton souci. De toute façon je vois mal poser un rapport de bug sur la 2.4.4 alors que la 2.6.0 est disponible. Il y a de grandes chances qu'ils te répondent de télécharger la dernière version.

Citation:

Envoyé par lg_53

Car il me semble, dans mes souvenirs du temps où ça marchait, j'étais en 1.9.qqch

Oui 1.9.4 sur une Debian installée depuis les DVD de base et jamais mise à jour (celle que j'utilise habituellement pour travailler). Ca a été mon premier essai indiqué dans mon premier post. Tu peux de ton côté essayer la même chose (tu te récupères une image VirtualBox comme on en trouve de partout). Et (accessoirement) j'ai tenté sur d'autres Linux qui, eux, sont mis à jour (Ubuntu, Xubuntu) mais ça reste quand-même 1.9.4. Il a fallu spécifiquement télécharger le source et l'installer manuellement pour avoir une version plus élevée (mais là alors trop élevée par rapport à toi).

les versions sur PiPy ne conviennent pas ?

Code:

1
2
3
C:\Users\user>pip install pylint==
ERROR: Could not find a version that satisfies the requirement pylint== (from versions: 0.15.2, 0.16.0, 0.18.0, 0.18.1, 0.19.0, 0.20.0, 0.21.0, 0.21.1, 0.21.2, 0.21.3, 0.22.0, 0.23.0, 0.24.0, 0.25.0, 0.25.1, 0.25.2, 0.26.0, 0.27.0, 0.28.0, 1.0.0, 1.1.0, 1.2.0, 1.2.1, 1.3.0, 1.3.1, 1.4.0, 1.4.1, 1.4.2, 1.4.3, 1.4.4, 1.4.5, 1.5.0, 1.5.1, 1.5.2, 1.5.3, 1.5.4, 1.5.5, 1.5.6, 1.6.0, 1.6.1, 1.6.2, 1.6.3, 1.6.4, 1.6.5, 1.7.0, 1.7.1, 1.7.2, 1.7.3, 1.7.4, 1.7.5, 1.7.6, 1.8.0, 1.8.1, 1.8.2, 1.8.3, 1.8.4, 1.9.0, 1.9.1, 1.9.2, 1.9.3, 1.9.4, 2.0.0.dev0, 2.0.0.dev1, 2.0.0.dev2, 2.0.0, 2.0.1, 2.1.0, 2.1.1, 2.2.0, 2.2.1, 2.2.2, 2.2.3, 2.3.0.dev0, 2.3.0.dev1, 2.3.0.dev2, 2.3.0, 2.3.1, 2.4.0, 2.4.1, 2.4.2, 2.4.3, 2.4.4, 2.5.0, 2.5.1, 2.5.2)
ERROR: No matching distribution found for pylint==

07/05/2020, 11h12
lg_53

Merci pour vos retours.

Alors fin mot de l'histoire :
J'ai updaté pylint via conda. Il a installé la 2.5.0 et le problème n'apparait plus.
Par contre quand je fais tourner Pyreverse sur tout mon code un autre bug apparait (qui n'apparaissait pas avant bien sûr!). Donc je suis reparti dans une identification d'un code minimal produisant le nouveau bug avec cette nouvelle version. Pour ceux que ça intéresse, j'ai ouvert un autre bug report du coup :

https://github.com/PyCQA/pylint/issues/3602

Merci à vous
07/05/2020, 13h35
Sve@r

Citation:

Envoyé par lg_53

...un autre bug apparait (qui n'apparaissait pas avant bien sûr!)

Je suis allé voir ton bugreport et récupéré ton exemple. Malheureusement le bug demeure présent sur la 2.6.0.
07/05/2020, 13h58
lg_53

Bah déjà si tu arrives aussi à le reproduire cela met à priori mon installation hors de cause. J'ai pas pris la 2.6 vu qu'elle n'est pas disponible via pip (ca s'arrete à 2.5.2, et j'obtiens le nouveau bug avec la 2.5.0 et la 2.5.2).

Donc bon j'espère que ce sera fixé, car ça m'était bien utile.