Skip to content

Commit c5dfe4e

Browse files
committed
incomplete commit because some bug tracking messed it all up
1 parent d04772a commit c5dfe4e

File tree

3 files changed

+7
-13
lines changed

3 files changed

+7
-13
lines changed

Runtime/Mapbox/BaseModule/Map/TileCreator.cs

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,6 @@ public class TileCreator : ITileCreator
1919
private readonly int UsingLinearColorspace = Shader.PropertyToID("_UsingLinearColorspace");
2020
private ObjectPool<UnityMapTile> _tilePool;
2121
private UnityContext _unityContext;
22-
private TerrainStrategy _terrainStrategy;
23-
private ElevationLayerProperties _settings;
2422
private int _cacheSize;
2523

2624
public UnityMapTile GetTile() => _tilePool.GetObject();
@@ -31,15 +29,12 @@ public TileCreator(UnityContext unityContext, TerrainStrategy terrainStrategy,
3129
{
3230
TileMaterials = tileMaterials;
3331
_unityContext = unityContext;
34-
_terrainStrategy = terrainStrategy;
35-
_settings = elevationProperties ?? new ElevationLayerProperties();
3632
_cacheSize = cacheSize;
3733
}
3834

3935
public IEnumerator Initialize()
4036
{
4137
_tilePool = new ObjectPool<UnityMapTile>(() => CreateTile(_unityContext));
42-
_terrainStrategy.Initialize(_settings);
4338
yield return _tilePool.InitializeItems(_cacheSize);
4439
}
4540

@@ -66,8 +61,6 @@ private UnityMapTile CreateTile(UnityContext unityContext)
6661
}
6762

6863
tile.gameObject.SetActive(false);
69-
70-
_terrainStrategy.RegisterTile(tile, false);
7164
return tile;
7265
}
7366
}

Runtime/Mapbox/ImageModule/Terrain/TerrainLayerModule.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,13 @@ public TerrainLayerModule(Source<TerrainData> source, TerrainLayerModuleSettings
2626
_settings = settings;
2727
_retainedTerrainTiles = new HashSet<CanonicalTileId>();
2828
_rasterSource = source;
29-
_terrainStrategy = new ElevatedTerrainStrategy();
29+
_terrainStrategy = new FlatTerrainStrategy(); //new ElevatedTerrainStrategy();
3030
}
3131

3232
public virtual IEnumerator Initialize()
3333
{
3434
yield return _rasterSource.Initialize();
35-
35+
_terrainStrategy.Initialize(_settings.ElevationLayerProperties);
3636
if(_settings.LoadBackgroundTextures)
3737
{
3838
_rasterSource?.DownloadAndCacheBaseTiles();
@@ -53,9 +53,10 @@ public virtual void LoadTempTile(UnityMapTile unityTile)
5353
for (int i = parentTileId.Z; i >= 2; i--)
5454
{
5555
parentTileId = parentTileId.Parent;
56-
if (_rasterSource.GetInstantData(parentTileId, out var instantData))
56+
if (_rasterSource.GetInstantData(parentTileId, out var instantData) && instantData.IsElevationDataReady)
5757
{
5858
unityTile.TerrainContainer.SetTerrainData(instantData, _settings.UseShaderTerrain, TileContainerState.Temporary);
59+
_terrainStrategy.RegisterTile(unityTile, !_settings.UseShaderTerrain);
5960
return;
6061
}
6162
}
@@ -73,15 +74,14 @@ public virtual bool LoadInstant(UnityMapTile unityTile)
7374
var targetTileId = GetDataId(unityTile.CanonicalTileId);
7475
if (_rasterSource.GetInstantData(targetTileId, out var instantData) && instantData.IsElevationDataReady)
7576
{
76-
_terrainStrategy.RegisterTile(unityTile, false);
77-
Debug.Log(unityTile.CanonicalTileId.Z + " using " + instantData.TileId.Z);
7877
unityTile.TerrainContainer.SetTerrainData(instantData, _settings.UseShaderTerrain);
78+
_terrainStrategy.RegisterTile(unityTile, !_settings.UseShaderTerrain);
7979
return true;
8080
}
8181

8282
return false;
8383
}
84-
84+
8585
public virtual bool RetainTiles(HashSet<CanonicalTileId> retainedTiles, Dictionary<UnwrappedTileId, UnityMapTile> activeTiles)
8686
{
8787
if (_settings.ElevationLayerType == ElevationLayerType.FlatTerrain)

Runtime/Mapbox/ImageModule/Terrain/TerrainLayerModuleSettings.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
using System;
22
using Mapbox.BaseModule.Data.Interfaces;
33
using Mapbox.BaseModule.Map;
4+
using Mapbox.ImageModule.Terrain.Settings;
45
using UnityEngine;
56

67
namespace Mapbox.ImageModule.Terrain

0 commit comments

Comments
 (0)