-
Notifications
You must be signed in to change notification settings - Fork 478
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
HITL: Make skinning support optional. #1791
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally, looks good!
Note: Might need to perform another pre-commit before before adding/committing to ensure ci test passes.
4c09cab
to
942933c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left some comments, otherwise LGTM
habitat-hitl/habitat_hitl/_internal/networking/keyframe_utils.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the case that skinning is disabled, are you doing anything to prevent this code from running on the client? https://github.com/0mdc/siro_hitl_unity_client/pull/4/files#diff-f071a636013728b2ff66c2068fe1e9cee3afc61b5af1534a2b6278794e8db51cR281
7beba80
to
d63b5ce
Compare
Yes. Skinned meshes with a |
91850f6
to
b41b209
Compare
* Pass skinning keyframe data to the client. * Add configuration to activate or deactivate skinning. * Formatting fix. * Add checks for rig removal from keyframes. * Refactoring. Remove conditional consolidation of rigs.
* Pass skinning keyframe data to the client. * Add configuration to activate or deactivate skinning. * Formatting fix. * Add checks for rig removal from keyframes. * Refactoring. Remove conditional consolidation of rigs.
* Pass skinning keyframe data to the client. * Add configuration to activate or deactivate skinning. * Formatting fix. * Add checks for rig removal from keyframes. * Refactoring. Remove conditional consolidation of rigs.
Motivation and Context
This changeset makes skinning transmission optional.
self._hitl_config.networking.active_features.skinning
is disabled, rig keyframe data is removed before transmission.See: 0mdc/siro_hitl_unity_client#4
How Has This Been Tested
Tested locally along with the Unity app.
Types of changes
Checklist