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

[mtoh] Add option to schedule updates through Qt instead of Maya. #757

Closed

Conversation

marsupial
Copy link
Contributor

@marsupial marsupial commented Aug 31, 2020

This PR addresses two outstanding issues:

  • The refresh on idle callback being done for every delegate separately, and incurring a bit of a penalty to get the update.
  • There's an outstanding issue in Maya where a user holding the mouse down button will block all updates....We'd like to think our users are going to be actively using their cursor and would still like visual feedback in those cases.

From offline conversations it seems 2 is in your court to solve, but this provides a work-around until then.
I does introduce Qt as a dependency into mtoh (which I think no-one really wants) but we need to get around the issue at hand.

-DMTOH_QT_UPDATES=OFF can be used to make mtoh use a modified version of the old code-path (using an MTimer callback, but that version moves away from MGlobal::executeOnIdle('refresh -f') in favor of M3dView::scheduleRefreshAllViews()

       Refactor the Maya implementation to have 1 timer/callback and avoid MGlobal::executeCommandOnIdle('refresh -f')
@kxl-adsk kxl-adsk added do-not-merge-yet Development is not finished, PR not ready for merge mtoh Related to legacy Maya to Hydra plugin. labels Sep 1, 2020
@kxl-adsk
Copy link

kxl-adsk commented Dec 7, 2021

This has been logged internally as MAYA-110426 as an improvement for Maya. I will close this PR.

@kxl-adsk kxl-adsk closed this Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge-yet Development is not finished, PR not ready for merge mtoh Related to legacy Maya to Hydra plugin.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants