Importance de l'ordre des imports dans Pyside

Version imprimable

16/04/2024, 09h20
Sve@r

Importance de l'ordre des imports dans Pyside
Bonjour à tous

J'ai eu hier un souci sur un code Pyside6. Ce code fait partie de ceux que j'ai mis en téléchargement dans mes exemples PyQt (donc ça veut dire qu'il avait fonctionné à un moment) mais hier il ne fonctionnait plus.

Et après décortication, j'ai trouvé que le souci provenait de l'ordre des imports.

Premier exemple
Code:

1 2 3 4 from PySide6.QtWidgets import * from PySide6.QtCore import * print(issubclass(QSlider, QWidget))
Ce code fonctionne et affiche "True" ce qui correspond à ce que j'attendais, un QSlider est bien un objet hérité d'un QWidget

Second exemple: on inverse les imports
Code:

1 2 3 4 from PySide6.QtCore import * from PySide6.QtWidgets import * print(issubclass(QSlider, QWidget))
Et là, ça ne va plus, il affiche "False" indiquant que là, un QSlider n'est pas un héritier d'un QWidget. De là (et c'était le souci de mon code) je ne peux plus le mettre dans un layout ou autre action inhérente à un QWidget.

Toutefois dans les deux cas, si je rajoute...
Code:

1 2 3 app=QApplication([]) s=QSlider() print(s)
...ça affiche bien Pyside6.QtWidgets.QSlider montrant que le QSlider provient bien du QtWidets. De même essayer d'importer un QSlider depuis QtCore ne fonctionne pas. Le QtCore ne possède pas de QSlider.

Donc avez-vous un avis sur le fait que dans le second cas, un QSlider n'est pas un QWidget alors que rien ne devrait le gêner à ce propos ?

Merci à tous.

PS: Qt 6.7.0, Pyside 6.7.0 et Python 3.10.12

Hello,

Si tu fais cela

Code:

1
2
3
4
5
import PySide6.QtCore as QtCore
import PySide6.QtWidgets as QtWidgets
 
 
print(issubclass(QtWidgets.QSlider, QtWidgets.QWidget))

ça donne quoi ?

16/04/2024, 12h41
Sve@r
Salut, merci de venir voir le sujet :ave:

Citation:

Envoyé par fred1599

ça donne quoi ?

Ca donne "True". On retombe sur un cas qui fonctionne. Après j'ai aussi la solution de bien checker l'ordre des imports ; ce qui fonctionne aussi. Mais ça ne m'explique pas ce qui se passe dans le second exemple 8-)

Pour info j'ai eu des soucis analogue en important QtGui avant QtWidgets. De là mes paramétrages visuels (ex QtAligment) ne fonctionnent plus (en fait ils sont refusés dans le setAlignement car eux-aussi ne sont pas du bon type). Il semble que QtWidgets doive explicitement être placé en premier (ensuite les deux autres ça semble moins problématique).

[edit]on peut modifier le second exemple ainsi...
Code:

1 2 from PySide6.QtCore import * from PySide6.QtWidgets import (QSlider, QWidget)
...et là aussi ça donne le bon résultat. En fait le souci semble venir de l'étoile à l'import.
16/04/2024, 13h00
fred1599

Citation:

Envoyé par Sve@r

Ca donne "True". On retombe sur un cas qui fonctionne.

Alors dans l'ordre des importations qui retournaient False, ça retourne True si on importe d'une certaine façon.

Je pense qu'en faisant from ... import * tu écrases certaines définitions importées du premier module (QtCore) par celles du second module (QtWidgets).

C'est à mon avis un cas atypique et assez complexe lié à des détails d'implémentation internes des modules PySide6.
16/04/2024, 13h08
Sve@r

Citation:

Envoyé par fred1599

C'est à mon avis un cas atypique et assez complexe lié à des détails d'implémentation internes des modules PySide6.

Ouaip. Il me semble que tu m'avais sermonné une fois sur le fait que tu n'aimais pas import * dans PyQt... :oops:

Je viens de commencer à traiter un premier petit projet (que j'avais fait en PyQt5 puis porté en PyQt6 et en PySide6) et j'ai remplacé tous les import * par des imports explicites et individuels. Là ça remarche (et l'ordre n'a plus d'importance) mais il va me falloir le faire maintenant de façon systématique :?
16/04/2024, 13h25
fred1599

Citation:

Envoyé par Sve@r

Il me semble que tu m'avais sermonné une fois sur le fait que tu n'aimais pas import * dans PyQt...

Oui c'est possible :mouarf: mais pas forcément pour cette raison...

Cette pratique peut rendre le code moins lisible car il devient difficile de déterminer quel nom vient de quel module. La PEP 8 conseille de privilégier des importations explicites pour maintenir la clarté du code.

Les bonnes pratiques c'est d'éviter le plus possible ce genre de notation.

bonjour

étrange ...

J'ai fait le test avec le pyside de mon système : 6.6.2 et ... pas de problème, même résultat (True) avec les 2 versions.

En venv avec 6.7 (qui vient de sortir), même problème que toi et bizarrement quel que soit l'ordre (et la version) j'ai avec print(QSlider.__mro__)

Code:

(<class 'PySide6.QtWidgets.QSlider'>, <class 'PySide6.QtWidgets.QAbstractSlider'>, <class 'PySide6.QtWidgets.QWidget'>, <class 'PySide6.QtCore.QObject'>, <class 'PySide6.QtGui.QPaintDevice'>, <class 'Shiboken.Object'>, <class 'object'>)

Note: avec 6.6 et 6.7 nous avons le même __init__.py, donc difficile de voir ce qui se passe.

---------------------
---------------------

Par contre, si je fais un print(len(QWidget.__subclasses__()), QWidget.__subclasses__()) alors l'ordre des 2 imports change tout 8O
ps: dans tous les cas, j'ai "PySide6.QtWidgets.QAbstractSlider"

False -> 18 classes (dérivées de QWidget)
True -> 28 classes (dérivées de QWidget)

ALORS QUE avec 6.6 quel que soit l'ordre d'import, j'ai : 27 classes (dérivées de QWidget)

---------------------
---------------------

EDIT:
Trouvé à https://wiki.qt.io/Qt_for_Python_Development_Notes
Code:

1 2 3 4 5 6 7 11. April 2024 * Bugs & Features PYSIDE-2675 Addressing the lazy import issues that were found in 6.7.0 : - Studying the possible solution, but it needs to consider other cases as well. - the *-import handling seems to be related, and affecting the mro content, duplicating some entries. - the type discovery handler might be related as well

Citation:

Envoyé par papajoker

ALORS QUE avec 6.6 quel que soit l'ordre d'import, j'ai : 27 classes (dérivées de QWidget)

En effet, j'ai commencé à avoir des soucis suite à une mise à jour.
Citation:
Envoyé par papajoker
Trouvé à https://wiki.qt.io/Qt_for_Python_Development_Notes
Code:

1 2 3 4 5 6 7 11. April 2024 * Bugs & Features PYSIDE-2675 Addressing the lazy import issues that were found in 6.7.0 : - Studying the possible solution, but it needs to consider other cases as well. - the *-import handling seems to be related, and affecting the mro content, duplicating some entries. - the type discovery handler might be related as well
Ok, c'est donc un bug. Merci de ton investissement :chin:

De mon côté j'ai commencé à reprendre mes codes et remplacer "*" par le détail des widgets utilisés (j'étais d'accord avec fred1599 la dernière fois qu'il en avait parlé mais j'avais la flemme, là ça m'a donné le coup de pied au luc qu'il me fallait).
Heureusement que pylint indique les imports inutiles parce qu'avec le copier/coller il y en avait pas mal de résiduels.

16/04/2024, 17h39
papajoker

Pour le nouveau "Lazy imports" de v6.7, qt en parle dans cette page "Lazy imports to improve the startup time"
Si le gain est si bon qu'ils le prétendent, alors pas de problème pour éradiquer les import * ;)