Skip to content

EMSUSD-3892 Integrate Debug Tools version 0.1.0-5#4659

Merged
seando-adsk merged 2 commits into
devfrom
barbalt/dev/retain-ui-size-dt
Jul 2, 2026
Merged

EMSUSD-3892 Integrate Debug Tools version 0.1.0-5#4659
seando-adsk merged 2 commits into
devfrom
barbalt/dev/retain-ui-size-dt

Conversation

@barbalt

@barbalt barbalt commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

PR Required for the Debug tools integration. It also adds code to retain some of the window column sizes

@barbalt barbalt self-assigned this Jun 30, 2026
pierrebai-adsk
pierrebai-adsk previously approved these changes Jun 30, 2026
Comment thread lib/usd/ui/debugTools/CompositionEditorCmd.cpp
Comment thread lib/usd/ui/renderSetup/renderSetupWindowCmd.cpp Outdated
@barbalt barbalt force-pushed the barbalt/dev/retain-ui-size-dt branch from 6f4d673 to 05e0a67 Compare June 30, 2026 18:48
@barbalt barbalt requested a review from pierrebai-adsk June 30, 2026 19:03
@barbalt barbalt changed the title Retain window size EMSUSD-3892 Integrate Debug Tools version 0.1.0-5 Jun 30, 2026
@barbalt barbalt assigned barbalt and unassigned barbalt Jun 30, 2026
@barbalt

barbalt commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator Author

All green with a custom internal pr using the internal branch with the latest version of the plugin.

@barbalt barbalt added the ready-for-merge Development process is finished, PR is ready for merge label Jun 30, 2026

@seando-adsk seando-adsk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

You need to sync your branch to latest dev branch and remove the change to renderSetupWindowCmd.cpp - that header file was moved below and put inside ifdef.

@seando-adsk seando-adsk removed the ready-for-merge Development process is finished, PR is ready for merge label Jul 2, 2026

@seando-adsk seando-adsk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I merged in latest dev to remove the change to renderSetupWindowCmd.cpp. I'll re-run the internal preflight for you.

@seando-adsk seando-adsk added ready-for-merge Development process is finished, PR is ready for merge shared_components Shared Components (Max / Maya) labels Jul 2, 2026
@seando-adsk

Copy link
Copy Markdown
Collaborator

This PR requires an updated version of the debug-tools shared library which is integrated from internal repo. I ran an preflight with that updated version and this PR and it passed. Marking as ready-for-merge.

@seando-adsk seando-adsk merged commit 9c4244e into dev Jul 2, 2026
10 checks passed
@seando-adsk seando-adsk deleted the barbalt/dev/retain-ui-size-dt branch July 2, 2026 15:33
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 shared_components Shared Components (Max / Maya)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants