Skip to content
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

[Merged by Bors] - Add synonyms for transform relative vectors #1667

Closed

Conversation

guimcaballero
Copy link
Contributor

Fixes #1663.

I think the directions are correct (same as here), but please double check because I might have mixed them up.

@alice-i-cecile alice-i-cecile added A-Core Common functionality for all bevy apps C-Usability A targeted quality-of-life change that makes Bevy easier to use labels Mar 16, 2021
@bitshifter
Copy link
Contributor

Note that this could use the constants added in #1674 if it lands.

@cart
Copy link
Member

cart commented Apr 12, 2021

We've tried this in the past (#1300) and reverted it because it was broken (#1465). I think this pr resolves the issue because it flips the "forward" direction to be -local_z. This appears to behave as expected (and godot also being right handed y-up also increases my confidence in these directions). When I test in 3d and 2d everything behaves as expected.

@cart
Copy link
Member

cart commented Apr 12, 2021

bors r+

bors bot pushed a commit that referenced this pull request Apr 12, 2021
Fixes #1663.

I think the directions are correct (same as [here](https://docs.godotengine.org/en/stable/classes/class_vector3.html?highlight=forward#constants)), but please double check because I might have mixed them up.

Co-authored-by: guimcaballero <guim.caballero@gmail.com>
Co-authored-by: Guim Caballero <guim.caballero@gmail.com>
@bors bors bot changed the title Add synonyms for transform relative vectors [Merged by Bors] - Add synonyms for transform relative vectors Apr 12, 2021
@bors bors bot closed this Apr 12, 2021
jihiggins pushed a commit to jihiggins/bevy that referenced this pull request Apr 18, 2021
Fixes bevyengine#1663.

I think the directions are correct (same as [here](https://docs.godotengine.org/en/stable/classes/class_vector3.html?highlight=forward#constants)), but please double check because I might have mixed them up.

Co-authored-by: guimcaballero <guim.caballero@gmail.com>
Co-authored-by: Guim Caballero <guim.caballero@gmail.com>
ostwilkens pushed a commit to ostwilkens/bevy that referenced this pull request Jul 27, 2021
Fixes bevyengine#1663.

I think the directions are correct (same as [here](https://docs.godotengine.org/en/stable/classes/class_vector3.html?highlight=forward#constants)), but please double check because I might have mixed them up.

Co-authored-by: guimcaballero <guim.caballero@gmail.com>
Co-authored-by: Guim Caballero <guim.caballero@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Core Common functionality for all bevy apps C-Usability A targeted quality-of-life change that makes Bevy easier to use
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add synonyms for transform relative vectors
4 participants