Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

handle namespace migration of QStringListModel between Maya 2019 and Maya 2020 in userExportedAttributesUI #812

Conversation

mattyjams
Copy link
Contributor

Maya 2020 and later use Qt/PySide2 5.12.5, where QStringListModel is exported to Python with QtCore, which is also where it lives in C++. Maya 2019 and earlier use Qt 5.6.1 and PySide2 2.0.0-alpha, where QStringListModel was incorrectly exported with QtGui.

See this bug for more detail:
https://bugreports.qt.io/browse/PYSIDE-614

This change addresses #366. I also took this opportunity to remove support for Maya 2016 and earlier.

Maya 2016 and earlier used PySide, while 2017 and later use PySide2.
…Maya 2020

Maya 2020 and later use Qt/PySide2 5.12.5, where QStringListModel is exported
to Python with QtCore, which is also where it lives in C++. Maya 2019 and
earlier use Qt 5.6.1 and PySide2 2.0.0-alpha, where QStringListModel was
incorrectly exported with QtGui.

See this bug for more detail:
https://bugreports.qt.io/browse/PYSIDE-614
Copy link
Collaborator

@seando-adsk seando-adsk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@kxl-adsk kxl-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Oct 6, 2020
@kxl-adsk kxl-adsk merged commit 6a13524 into Autodesk:dev Oct 6, 2020
@mattyjams mattyjams deleted the pr/fix_QStringListModel_in_userExportedAttributesUI branch October 6, 2020 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pxr Related to Pixar plugin ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants