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

changed minus sign to a plus sign #1316

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

izi2bhappy
Copy link

Israelkariti

Mar 21, 2:41 PM PDT

In class: "RotateWithLocationProvider"
In method "Vector3 getNewEulerAngles(float newAngle)"
Code at line 152 : "euler.y = -newAngle;"
causing the player game object to turn CCW when the android device is actully turning CW (and vice versa)
should be : "euler.y = newAngle;"

Related issue

Example: Closes #832. Relates to #832.

Description of changes

Your text here!

QA checklists

  • Add relevant code comments. Every API class and method should have <summary> description as well as description of parameters.
  • Add tests for new/changed/updated classes and methods!!!
  • Check out conventions in CONTRIBUTING.md.
  • Check out conventions in CODING-STYLE.md
  • Update the changelog
  • Update documentation.

Reviewers

Tag your reviewer(s). Choose wisely.

	
Israelkariti

Mar 21, 2:41 PM PDT

In class: "RotateWithLocationProvider" 
In method "Vector3 getNewEulerAngles(float newAngle)"
Code at line 152 : "euler.y = -newAngle;"
causing the player game object to turn CCW when the android device is actully turning CW (and vice versa)
should be : "euler.y = newAngle;"
from 'TileJSONReponse' to 'TileJSONResponse'
@izi2bhappy
Copy link
Author

Fixed typo in file name

@brnkhy
Copy link
Contributor

brnkhy commented Sep 13, 2019

@atripathi-mb @jordy-isaac I'm not able to test this as it requires an android device (device rotation etc), can you give it a shot?

@abhishektrip
Copy link
Contributor

@brnkhy It can be tested with any device.

@ugacode
Copy link

ugacode commented Oct 4, 2020

Unclear to me why this hasn't been merged in by now, as someone who is trying to use Mapbox for the first time, struggling with this over a long period of time sours an otherwise positive experience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

How to request exactly all tiles in Camera View ?
4 participants