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

Calculating transform parameters instead of overriding worldToCameraMatrix #159

Merged
merged 1 commit into from
Mar 9, 2018
Merged

Calculating transform parameters instead of overriding worldToCameraMatrix #159

merged 1 commit into from
Mar 9, 2018

Conversation

delapuente
Copy link
Contributor

@delapuente delapuente commented Feb 26, 2018

Fix #158. In addition, it advances in some of the efforts in #154 . Now, like Unity, it overrides the values of the transform component, so it can not be changed manually and it can continue affecting its children. The main/central camera can be considered the head of the player so things can be positioned relative to the player's head (useful for GUIs).

@delapuente delapuente requested a review from cvan February 26, 2018 12:23
@caseyyee
Copy link
Contributor

caseyyee commented Feb 27, 2018

@delapuente I gave this a whirl.

Seems to work fine when I move the camera prefab location around the scene. Nice work!

Looks like we'll need to apply the same thing to the controller positions as well. I guess it makes sense to have the controllers as a child object of the camera prefab.

@delapuente
Copy link
Contributor Author

delapuente commented Feb 27, 2018

Thank you very much @caseyyee I open #162 to discuss how to manage controllers.

@delapuente
Copy link
Contributor Author

delapuente commented Mar 5, 2018

@cvan @caseyyee do you think we can merge this and improve towards v1.1?

@caseyyee
Copy link
Contributor

caseyyee commented Mar 6, 2018

@delapuente looks good, but the offset controller needs to be solved along with this. I'll make a complementary patch that we can land it together. otherwise looks great!

trs.GetColumn(1).magnitude,
trs.GetColumn(2).magnitude
);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

indentation

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