Skip to content
This repository has been archived by the owner on Aug 11, 2024. It is now read-only.

cleaned up sdk a bit #576

Merged
merged 6 commits into from
May 9, 2020
Merged

cleaned up sdk a bit #576

merged 6 commits into from
May 9, 2020

Conversation

StephenHodgson
Copy link
Contributor

@StephenHodgson StephenHodgson commented May 8, 2020

XRTK - Mixed Reality Toolkit Change Request

Overview

  • Added docs and simplified interfaces.
  • Added Core system validation methods so classes can wait until the system becomes available.
  • Moved the GazeProvider from the SDK into Core because it's a required component for the Input System and should be shipped with the default Input System implementation.

Breaking Changes:

  • renamed IMixedRealityControllerVisualizer.GameObjectProxy -> IMixedRealityControllerVisualizer.GameObject
  • renamed IMixedRealityGazeProvide.GameObjectReference -> IMixedRealityGazeProvider.GameObject
  • moved XRTK.SDK.Input.GazeProvider -> XRTK.Services.InputSystem.GazeProvider

Submodule Changes

XRTK/com.xrtk.sdk#178

Added docs and simplified interfaces
@StephenHodgson StephenHodgson added Ready for review PR finished primary development, open for review and removed In Progress PR currently still being developed labels May 8, 2020
@StephenHodgson StephenHodgson merged commit 410255e into development May 9, 2020
@StephenHodgson StephenHodgson deleted the dev/sdk-cleanup branch May 9, 2020 09:53
XRTK-Build-Bot pushed a commit that referenced this pull request Dec 25, 2020
* cleaned up sdk a bit

Added docs and simplified interfaces

* Moved GazeProvider into core

cleaned up headers and references

* updated unit tests
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Breaking Change Ready for review PR finished primary development, open for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants