mapbox / mapbox-unity-sdk

Mapbox Unity SDK - https://www.mapbox.com/unity/
Other
725 stars 210 forks source link

Tile-Texture Preview representation #1847

Closed Markovicho closed 2 years ago

Markovicho commented 2 years ago

image

Hey @brnkhy :-)

Since we are using more customized shading on our tiles it's getting more obvious that the preview of loading tiles is visualized by somewhat like an pretty low zoom-level texture showing whole europe. This tile seems to be hardcoded as it's also appearing if we clear the tile cache and never load a tile like that.

I checked out other map services and came to the conclusion that they are keeping the last known texture (stretched / blurry) while the new texutre hasn't arrived.

Is this something which will be sorted out ? Otherwise it would be nice to get an insight on what script we could manipulate to force another loading tile. This could sort of our visual glitch for now :-)

by the way we are using latest commit from performance branch.

best regards and keep up the really solid work !

Mark

brnkhy commented 2 years ago

Hey @Markovicho I'm afraid I don't understand the issue. when the system is loading new tiles, it uses a temporarily textures, generally with a much lower quality. and are you having with this temp texture on custom layers? scene view in your first screenshot looks bugged for example. Street level image and that one below; that's definitely a bug. does it work fine when you use satellite imagery for example? or how did you set up custom layer? I'll need some more data here :)

Markovicho commented 2 years ago

Hey @brnkhy ,

i think we figured out what goes wrong and why it goes wrong by design :-)

image

The reason why our custom shader is showing the glitch mentioned in the first post is that we ignored the tiling and displayed the whole texture...

The fact that the tiling of the tile-textures differs between the loading situation should be documented. I think a lot of people writing their own shaders could run into this issue.

anyways thx @brnkhy