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

Add support for display of VtArray<bool> to Hydra scene browser #187

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

ppt-adsk
Copy link
Collaborator

@ppt-adsk ppt-adsk commented Oct 15, 2024

Very useful to debug instancer masking.

@ppt-adsk ppt-adsk requested a review from debloip-adsk October 15, 2024 15:49
@@ -92,6 +92,13 @@ foreach(SOURCE IN ITEMS ${SOURCES})
set(SET_EXPANDED_REPLACEMENT "setExpanded\(true\);")
string(REGEX REPLACE "${SET_EXPANDED_REGEX}" "${SET_EXPANDED_REPLACEMENT}" FILE_CONTENTS "${FILE_CONTENTS}")

# Patch in support for VtArray<bool>. Arbitrarily place it after support
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Somewhat ugly to do this in a loop that goes over all files, but convenient.

@ppt-adsk ppt-adsk self-assigned this Oct 15, 2024
Copy link
Collaborator

@debloip-adsk debloip-adsk left a comment

Choose a reason for hiding this comment

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

Seems like a good candidate for a PR to Pixar as well, do we/should we have a ticket for that?

@ppt-adsk ppt-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Oct 15, 2024
@ppt-adsk
Copy link
Collaborator Author

Seems like a good candidate for a PR to Pixar as well, do we/should we have a ticket for that?

I have just created HYDRA-1252.

@debloip-adsk debloip-adsk merged commit 5f73572 into dev Oct 15, 2024
10 checks passed
@debloip-adsk debloip-adsk deleted the tremblp/hydra_scene_browser_array_of_bool_support branch October 15, 2024 17:24
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.

2 participants