Skip to content

Conversation

wilhelmberg
Copy link
Contributor

Related issue

ref #787

Description of changes

Fix wrong results for nautical miles because of unwanted integer division.
Also add a test for calculation of nautical miles.

@atripathi-mb

Copy link
Contributor

@abhishektrip abhishektrip left a comment

Choose a reason for hiding this comment

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

looks good.

@abhishektrip abhishektrip merged commit 48f77ae into develop May 18, 2018
abhishektrip added a commit that referenced this pull request May 18, 2018
* develop:
  fix a calculation error in uv modifier that occasionally breaks roof uvs (#754)
  fix #787 NauticalMiles unit is wrong (#788)
@wilhelmberg wilhelmberg deleted the cheapruler-nauticalmiles branch May 22, 2018 06:42
abhishektrip added a commit that referenced this pull request May 24, 2018
* develop:
  Styling improvement (#718)
  GoogleARCore fix for Unity@2018  (#791)
  Poi scale fix (#799)
  Replace hero buildings (#797)
  fix a bug where terrain data causing IndexOutOfRangeException (#795)
  fix a calculation error in uv modifier that occasionally breaks roof uvs (#754)
  fix #787 NauticalMiles unit is wrong (#788)

# Conflicts:
#	sdkproject/Assets/Mapbox/Examples/0_PrefabScenes/CitySimulator.unity
#	sdkproject/Assets/Mapbox/Resources/MapboxStyles/Materials/MapboxStylesFacadesPerRenderer.mat
#	sdkproject/Assets/Mapbox/Resources/MapboxStyles/Styles/MapboxSampleStyles/Realistic.meta
#	sdkproject/ProjectSettings/ProjectVersion.txt
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.

2 participants