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

Transfrom controller pitch angle to match Focus controllers in system. #1363

Merged
merged 1 commit into from
Jul 3, 2019

Conversation

daoshengmu
Copy link
Contributor

@daoshengmu daoshengmu commented Jul 3, 2019

Per #1354, we want to make Focus Plus controller's position match the controllers in system.

@daoshengmu
Copy link
Contributor Author

I can't find a way to get their system version to decide if we need to make this change on their previous system. I might need to get assistance from the partner.

Copy link
Contributor

@bluemarvin bluemarvin left a comment

Choose a reason for hiding this comment

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

We either need to get the updated model off of the device or apply this transform to the model directly. We shouldnot be applying this transform twice every frame.

@bluemarvin
Copy link
Contributor

I can't find a way to get their system version to decide if we need to make this change on their previous system. I might need to get assistance from the partner.

That's probably fine. I doubt there will be very many people running the old system.

@daoshengmu daoshengmu force-pushed the adjustFocusControllerTransform branch from 09847d7 to 8d9a52c Compare July 3, 2019 22:30
@daoshengmu daoshengmu self-assigned this Jul 3, 2019
@bluemarvin bluemarvin merged commit c2dfa60 into master Jul 3, 2019
@bluemarvin bluemarvin deleted the adjustFocusControllerTransform branch July 3, 2019 23:29
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.

2 participants