Skip to content

Fix for Maya 2019 error#5

Merged
j0yu merged 3 commits into
masterfrom
fix-maya-2019-quintptr
Jun 2, 2020
Merged

Fix for Maya 2019 error#5
j0yu merged 3 commits into
masterfrom
fix-maya-2019-quintptr

Conversation

@j0yu

@j0yu j0yu commented May 28, 2020

Copy link
Copy Markdown

See also PYSIDE-1063 and shotgunsoftware#95

Screen grabbing in tk-multi-snapshot causes the following error:

# Error: TypeError: file <maya console> line 1: 'PySide2.QtGui.QScreen.grabWindow' called with wrong argument types:
  PySide2.QtGui.QScreen.grabWindow(int, int, int, int, int)
Supported signatures:
  PySide2.QtGui.QScreen.grabWindow(quintptr, int = 0, int = 0, int = -1, int = -1) # 

Not an issue in Maya 2020 which uses a more recent PySide2/Qt version


Fixes

  • Old PySide2 quintptr type found in Maya 2019, see also PYSIDE-1063

@j0yu j0yu requested a review from Liametc May 28, 2020 17:31
@j0yu j0yu self-assigned this May 28, 2020
@j0yu j0yu marked this pull request as ready for review June 1, 2020 16:40
@j0yu j0yu changed the title WIP: Fix for Maya 2019 error Fix for Maya 2019 error Jun 1, 2020
@j0yu j0yu force-pushed the fix-maya-2019-quintptr branch from 37ced80 to 1268a34 Compare June 2, 2020 10:03
@j0yu

j0yu commented Jun 2, 2020

Copy link
Copy Markdown
Author

Gonna split into 2 PRs, I'll automatically merge in the follow-up one

@j0yu j0yu merged commit e54da65 into master Jun 2, 2020
@j0yu j0yu deleted the fix-maya-2019-quintptr branch June 2, 2020 10:07
j0yu pushed a commit that referenced this pull request Jun 2, 2020
* Initial fix

* Py3 friendly long

* Fixed missing :, extra comment space
j0yu pushed a commit that referenced this pull request Nov 5, 2020
* Initial fix

* Py3 friendly long

* Fixed missing :, extra comment space
j0yu pushed a commit that referenced this pull request Mar 10, 2021
* Initial fix

* Py3 friendly long

* Fixed missing :, extra comment space
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants