8000 update lumin hand controller data provider by StephenHodgson · Pull Request #51 · XRTK/com.xrtk.lumin · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on May 13, 2022. It is now read-only.

update lumin hand controller data provider #51

Merged
merged 2 commits into from
Mar 30, 2020

Conversation

StephenHodgson
Copy link
Contributor

Continued from XRTK/com.xrtk.core#476

@StephenHodgson StephenHodgson requested a review from FejZa March 29, 2020 19:13
Copy link
Contributor
@SimonDarksideJ SimonDarksideJ left a comment

Choose a reason for hiding this comment

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

Same comment as Oculus, fine with updates but we shouldn't be using a separate data provider for hands

Copy link
Contributor
@FejZa FejZa left a comment

Choose a reason for hiding this comment

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

Approved but I'll have to verify all of these changes whether things still work as expected.

@FejZa FejZa merged commit 92d85f4 into feature/hand-tracking Mar 30, 2020
@FejZa FejZa deleted the change-requests/hand-tracking branch March 30, 2020 12:06
StephenHodgson added a commit that referenced this pull request Apr 7, 2020
* Add Lumin hand controller data provider

* Fix base class

* Implement hand data provider interface

* Remove obsolete lumin platform hand controller data provider

* Create controller stub

* Remove need for hand controller data provider profile

* Revert Lumin speciifc hand controller

* Implement Lumin data provider profile

* Implement Lumin data provider profile inspector

* Add XRTK assembly reference to fix inspector missing assembly issue

* Fix deprecated Lumin controller API

* Implement Lumin hands init / deinit

* Implement hand data conversion

* Implement is tracking enabled flag for Lumin

* Add comment for missing rotation

* Fix Lumin dependent compilation

* Use LateUpdate for hand update instead of Update

* Fix Unity serialization error when selecting Lumin profile asset when not targeting Lumin

* Implement Lumin profile lock using new Utility

* Update profile platform utility usage

* Resolve merge issue

* Fix platform flag for Lumin inspector

* Set file icons

* Create Lumin hand controller and mapping profile + inspector

* Add required RefreshActiveControllers override to build

* Removed base hand data provider

* Move hand controller detection into existin lcdp

* Adjust Lumin to recent changes

* Bring back lumin hand controller data provider profile

* Move LuminControllerDataProvider constructor to top

* Reset Lumin data provider controller

* Update hand controller data provider to fit recent changes

* Remove hand tracking enabled setting (disable by removing data provider)

* Update convert to set all missing joints zero fo rnow

* Introduce shared hand profile configuraiton

* Introduce base hand controller data provide to consume shared profile

* Add label to Lumin specific in inspector

* Introduce HandMeshingEnabled profile setting

* Introduce "Hand Tracking System Profile"

* Remove package dependency to legacy magic leap package

* update lumin hand controller data provider (#51)

* update lumin hand controller data provider

* updated base controller data provider profile namespace

* fixed compiler errors from platform changes

* Fusion Lumin controller dp profiles

* updated inspector

* bumped minor version

* Remove hand ray type setting

* Change requests/hand tracking (#54)

* updated references

* updated references and cleaned up a bit

* moved LuminHandDataConverter to Lumin.Utilities

* sorted usings

* fixed hand data provider inspector

* Restore LuminHandControllerDataProvider

* Restore lumin package dependency

* fixed compiler errors

* updated lumin hand controller provider profile

* added a check if configuration was applied

misc formatting

* fix compiler errors

* remove hand calls from controller data provider

* Fixed lumin hand controller support (#55)

* Fixed lumin hand controller support

updated palm pose

* only remove from registry if flag set

Co-authored-by: Stephen Hodgson <hodgson.designs@gmail.com>
Co-authored-by: Stephen Hodgson <StephenHodgson@users.noreply.github.com>
XRTK-Build-Bot pushed a commit that referenced this pull request Dec 25, 2020
* Add Lumin hand controller data provider

* Fix base class

* Implement hand data provider interface

* Remove obsolete lumin platform hand controller data provider

* Create controller stub

* Remove need for hand controller data provider profile

* Revert Lumin speciifc hand controller

* Implement Lumin data provider profile

* Implement Lumin data provider profile inspector

* Add XRTK assembly reference to fix inspector missing assembly issue

* Fix deprecated Lumin controller API

* Implement Lumin hands init / deinit

* Implement hand data conversion

* Implement is tracking enabled flag for Lumin

* Add comment for missing rotation

* Fix Lumin dependent compilation

* Use LateUpdate for hand update instead of Update

* Fix Unity serialization error when selecting Lumin profile asset when not targeting Lumin

* Implement Lumin profile lock using new Utility

* Update profile platform utility usage

* Resolve merge issue

* Fix platform flag for Lumin inspector

* Set file icons

* Create Lumin hand controller and mapping profile + inspector

* Add required RefreshActiveControllers override to build

* Removed base hand data provider

* Move hand controller detection into existin lcdp

* Adjust Lumin to recent changes

* Bring back lumin hand controller data provider profile

* Move LuminControllerDataProvider constructor to top

* Reset Lumin data provider controller

* Update hand controller data provider to fit recent changes

* Remove hand tracking enabled setting (disable by removing data provider)

* Update convert to set all missing joints zero fo rnow

* Introduce shared hand profile configuraiton

* Introduce base hand controller data provide to consume shared profile

* Add label to Lumin specific in inspector

* Introduce HandMeshingEnabled profile setting

* Introduce "Hand Tracking System Profile"

* Remove package dependency to legacy magic leap package

* update lumin hand controller data provider (#51)

* update lumin hand controller data provider

* updated base controller data provider profile namespace

* fixed compiler errors from platform changes

* Fusion Lumin controller dp profiles

* updated inspector

* bumped minor version

* Remove hand ray type setting

* Change requests/hand tracking (#54)

* updated references

* updated references and cleaned up a bit

* moved LuminHandDataConverter to Lumin.Utilities

* sorted usings

* fixed hand data provider inspector

* Restore LuminHandControllerDataProvider

* Restore lumin package dependency

* fixed compiler errors

* updated lumin hand controller provider profile

* added a check if configuration was applied

misc formatting

* fix compiler errors

* remove hand calls from controller data provider

* Fixed lumin hand controller support (#55)

* Fixed lumin hand controller support

updated palm pose

* only remove from registry if flag set

Co-authored-by: Stephen Hodgson <hodgson.designs@gmail.com>
Co-authored-by: Stephen Hodgson <StephenHodgson@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0