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

Problems with terrainExaggeration parameter in version > v.1.16 #3676

Closed
jacmendt opened this issue Mar 7, 2016 · 6 comments
Closed

Problems with terrainExaggeration parameter in version > v.1.16 #3676

jacmendt opened this issue Mar 7, 2016 · 6 comments

Comments

@jacmendt
Copy link

jacmendt commented Mar 7, 2016

I expected some problems with the the terrainExaggeration parameter in cesium versions >= 1.17. I used a terrainExaggeration value of 4.0 in version 1.16 and it works for me. In newer versions, my terrain / imagery isn't display anymore if zooming to higher zoom levels. My area of interest was europe.

I reproduced the error running the sandbox application with this code:

var viewer = new Cesium.Viewer('cesiumContainer', {
    terrainExaggeration: 4.0
});

var terrainProvider = new Cesium.CesiumTerrainProvider({
    url : '//assets.agi.com/stk-terrain/world'
});
viewer.terrainProvider = terrainProvider;
@mramato
Copy link
Contributor

mramato commented Mar 7, 2016

Thanks @jacmendt, this is a known issue that we definitely plan on fixing for our next release (April 1st).

Since this is a duplicate of #3607, I'm going to close this issue. Thanks again.

@pjcozzi
Copy link
Contributor

pjcozzi commented Mar 21, 2016

@jacmendt this was fixed in #3734 and will be released in Cesium 1.20 on April 1.

@jacmendt
Copy link
Author

jacmendt commented Apr 6, 2016

@pjcozzi i tested today the new Cesium version 1.20 and the error still exists. The tile loading seems to crash at the greenwich meridian in case of the using of terrainExaggeration > 1. I also tested again the sandbox application described above and could reproduced the error, but I'm not sure if the sandbox application is also using the latest cesium version.

Btw. thanks for the help :)

@mramato
Copy link
Contributor

mramato commented Apr 6, 2016

I can confirm that this is still an issue, but only in specific portions of the Earth (which is why it was probably missed). Use the original sample code and zoom to Dancy, Eure-et-Loir, France in the Geocoder and you'll see it.

@bagnell can you take a look at this?

@pjcozzi
Copy link
Contributor

pjcozzi commented Apr 6, 2016

Thanks for the report, we have a few other things on our plate, but we'll try to look at this before the next release.

@pjcozzi
Copy link
Contributor

pjcozzi commented Apr 27, 2016

@bagnell can you take a quick look to see if this is small enough in scope to make 1.21 on Monday?

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

No branches or pull requests

3 participants