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

EMSUSD-988 Maya node observer classes #3590

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

pierrebai-adsk
Copy link
Collaborator

Add new classes to observe Maya nodes of a given type or a given Maya node. The observer forward changes to registered listeners. The goal is to standardize how a node is observed and to share the notification between multiple listeners.

This will be used by multiple classes in a subsequent pull request: MayaUsdProxyShapeBase, UsdStageMap and MayaSessionState.

The MayaNodeTypeObserver allows listening to new nodes being created or removed. The MayaNodeObserver allows listening to the node being renamed or reparented. Many use cases need to track the DAG path, UFE path or USD path of objects, mostly the USD stage itself.

@pierrebai-adsk pierrebai-adsk self-assigned this Feb 2, 2024
@pierrebai-adsk pierrebai-adsk added enhancement New feature or request adsk Related to Autodesk plugin labels Feb 2, 2024
Add new classes to observe Maya nodes of a given type or a given Maya node. The
observer forward changes to registered listeners. The goal is to standardize how
a node is observed and to share the notification between multiple listeners.

This will be used by multiple classes in a subsequent pull request:
MayaUsdProxyShapeBase, UsdStageMap and MayaSessionState.

The MayaNodeTypeObserver allows listening to new nodes being created or
removed. The MayaNodeObserver allows listening to the node being renamed or
reparented. Many use cases need to track the DAG path, UFE path or USD path of
objects, mostly the USD stage itself.
@pierrebai-adsk
Copy link
Collaborator Author

The only failures are three 2023 PF failed to launch, but otherwise every passed. Since this is new code not even used, it should not black merging.

The new code is used in the next PR (#3591_ which depends on this one and is working.

@pierrebai-adsk pierrebai-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Feb 5, 2024
@seando-adsk seando-adsk merged commit f25d18d into dev Feb 6, 2024
13 of 15 checks passed
@seando-adsk seando-adsk deleted the bailp/EMSUSD-988/maya-node-observers branch February 6, 2024 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adsk Related to Autodesk plugin enhancement New feature or request 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.

3 participants