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

Internal References Test #83

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mati-nvidia
Copy link
Contributor

This is still WIP. Just sharing to further discuss.

@asluk
Copy link

asluk commented Feb 27, 2024

curious as to what the Pcp.PrimIndex.DumpToDotGraph output looks like for the shader prims here... I'll try to scratch out some time to try it

@asluk
Copy link

asluk commented Feb 27, 2024

curious as to what the Pcp.PrimIndex.DumpToDotGraph output looks like for the shader prims here... I'll try to scratch out some time to try it

@chrizzFTD , @noWookies -- might be cool to inspect this dataset in Grill? 👍🏼 https://github.com/thegrill/grill

@asluk
Copy link

asluk commented Feb 28, 2024

curious as to what the Pcp.PrimIndex.DumpToDotGraph output looks like for the shader prims here... I'll try to scratch out some time to try it

@chrizzFTD , @noWookies -- might be cool to inspect this dataset in Grill? 👍🏼 https://github.com/thegrill/grill

here's the output from Grill--

image

it confirms that the Prefab hierarchy on this side of the external reference is not in the composition graph for Bolt_01, because it's the Prefab on the other side of the external reference that is in the composition graph for Bolt_01.

@meshula
Copy link
Contributor

meshula commented Sep 13, 2024

@mati-nvidia What's the current state of this one?

@mati-nvidia
Copy link
Contributor Author

I haven't had a chance to pick this back up yet, but I think all that's left to do is add some explanation based on Aaron's debugging.

I was going to add an example using specializes or inherits, but maybe that's out of scope for this asset and just a short explanation will suffice. It could link to another specializes/inherits asset in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants