OpenUSD icon indicating copy to clipboard operation
OpenUSD copied to clipboard

Documentation: Collections erroneously claim that you cannot mix collection configurations

Open dgovil opened this issue 1 year ago • 4 comments

The Collections and Patterns documentation claims that Note that you cannot mix collection configuration. We believe this is incorrect and this note should be removed to align with current USD behaviour.

We find that this is note is not true in USD 25.2. USD 25.2 allows for the mixing of these two types.

A sample setup is included here to reproduce the output below. test.py will generate the JSON, whereas diff.py will create a diff output. collectionsTest.zip

The output here is generated between USD 24.3 and USD 25.2, and shows how the results of the collection have changed. This shows the difference in collection behaviour between the two versions. I did not test versions in between. We prefer the new behaviour for what it's worth.

- means the item was in 24.3 but not in 25.2. + means it is new in 25.2.

/cornell/collections/root_expression
	- /


/cornell/collections/root_includes
	- /


/cornell/collections/root_excludes
	+ /cornell/assets
	+ /cornell/collections/root_includes
	+ /cornell/assets/left
	+ /cornell/lookdev/green
	+ /cornell/assets/right
	+ /cornell/collections
	+ /cornell/assets/top
	+ /cornell/collections/expression_includes_excludes
	+ /cornell/lookdev/grey
	+ /cornell/lookdev/grey/shader
	+ /cornell/collections/root_excludes
	+ /cornell/lookdev/red/shader
	+ /cornell/lookdev
	+ /cornell/assets/back
	+ /cornell
	+ /cornell/collections/expression_excludes
	+ /cornell/assets/bottom
	+ /cornell/lookdev/red
	+ /cornell/camera
	+ /cornell/assets/rectlight
	+ /cornell/lookdev/green/shader
	+ /cornell/collections/root_includes_excludes
	+ /cornell/collections/root_expression
	+ /cornell/assets/right_box
	+ /cornell/collections/expression_includes


/cornell/collections/root_includes_excludes
	+ /cornell/collections/root_includes
	+ /cornell/lookdev/green
	+ /cornell/collections
	+ /cornell/collections/expression_includes_excludes
	+ /cornell/lookdev/grey
	+ /cornell/lookdev/grey/shader
	+ /cornell/collections/root_excludes
	+ /cornell/lookdev/red/shader
	+ /cornell/lookdev
	+ /cornell
	+ /cornell/collections/expression_excludes
	+ /cornell/lookdev/red
	+ /cornell/camera
	+ /cornell/lookdev/green/shader
	+ /cornell/collections/root_includes_excludes
	+ /cornell/collections/root_expression
	+ /cornell/collections/expression_includes


/cornell/collections/expression_excludes
	- /cornell/collections/root_includes
	- /cornell/lookdev/green
	- /cornell/collections
	- /cornell/collections/expression_includes
	- /cornell/collections/expression_includes_excludes
	- /cornell/lookdev/grey
	- /cornell/lookdev/grey/shader
	- /cornell/collections/root_excludes
	- /cornell/lookdev
	- /cornell/collections/expression_excludes
	- /cornell/lookdev/red
	- /cornell/lookdev/green/shader
	- /cornell/collections/root_includes_excludes
	- /cornell/collections/root_expression
	- /cornell/lookdev/red/shader

dgovil avatar Feb 20 '25 23:02 dgovil

Hi @dgovil , to the best of our knowledge, the documentation is correct, and if both are authored on a prim, only the relationship-based description will be consulted. There were, however, a series of bug fixes throughout 2024 that resulted in membershipExpressions making more matches than they previously did.

spiffmon avatar Feb 21 '25 00:02 spiffmon

Ah! Consulting @gitamohr , there is an open bug that is likely responsible for what you are seeing. A fix is IP.

spiffmon avatar Feb 21 '25 00:02 spiffmon

Ah so the current behaviour is incorrect. That's good to know. Thanks.

dgovil avatar Feb 21 '25 01:02 dgovil

Filed as internal issue #USD-10689

(This is an automated message. See here for more information.)

jesschimein avatar Feb 21 '25 18:02 jesschimein