Skip to content

Comments

Fixed hang on exit of isolate select with geom subsets, with test.#234

Merged
lilike-adsk merged 3 commits intodevfrom
tremblp/HYDRA-1242/isolate_select_hang
Jan 17, 2025
Merged

Fixed hang on exit of isolate select with geom subsets, with test.#234
lilike-adsk merged 3 commits intodevfrom
tremblp/HYDRA-1242/isolate_select_hang

Conversation

@ppt-adsk
Copy link
Collaborator

@ppt-adsk ppt-adsk commented Dec 17, 2024

When a point instance prototype has a geom subset under a mesh, exiting isolate select causes a hang in Hydra Storm. Fixed by avoiding setting visibility off on geom subsets, which is irrelevant anyways for isolate select.

@ppt-adsk ppt-adsk self-assigned this Dec 17, 2024
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Without the fix, this test hangs. With the fix, it passes.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

All references to Maya 2026 removed.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Test data taken from HYDRA-1242 ticket, belongs to Autodesk.

@ppt-adsk ppt-adsk assigned ppt-adsk and unassigned ppt-adsk Jan 16, 2025
@ppt-adsk ppt-adsk added ready-for-merge Development process is finished, PR is ready for merge and removed do-not-merge-yet labels Jan 16, 2025
@lilike-adsk lilike-adsk merged commit eb243e9 into dev Jan 17, 2025
10 checks passed
@lilike-adsk lilike-adsk deleted the tremblp/HYDRA-1242/isolate_select_hang branch January 17, 2025 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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