Physics: Add support for merging collision groups. #1947
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When merging assets from multiple sources, it's desirable in some scenarios that we wish to merge physics collision groups. This adds support for optional user-provided identifiers which allow such merging. We also add an option to invert the selection of filtered groups, which can be used to create colliders which opt-in to collisions, rather than the default behaviour of having to opt-out. Additionally, added a utility function which will calculate a table indicating if two groups collide as well as tests for said utility. This utility can be used in authoring use-cases to provide feedback for content creators as well as providing a reference which can be used to validate an implementation.
|
Filed as internal issue #USD-7490 |
|
The PR is merged to the dev branch, with couple of minor updates: a024497. Thanks |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of Change(s)
When merging assets from multiple sources, it's desirable in some
scenarios that we wish to merge physics collision groups. This adds
support for optional user-provided identifiers which allow such
merging. We also add an option to invert the selection of filtered
groups, which can be used to create colliders which opt-in to
collisions, rather than the default behaviour of having to opt-out.
Additionally, added a utility function which will calculate a table
indicating if two groups collide as well as tests for said utility. This
utility can be used in authoring use-cases to provide feedback for
content creators as well as providing a reference which can be used to
validate an implementation.
Fixes Issue(s)