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

Translucent geometry not rendering as well as it used to? #7364

Closed
mramato opened this issue Dec 3, 2018 · 5 comments · Fixed by #7371
Closed

Translucent geometry not rendering as well as it used to? #7364

mramato opened this issue Dec 3, 2018 · 5 comments · Fixed by #7371

Comments

@mramato
Copy link
Contributor

mramato commented Dec 3, 2018

Somewhere between 1.50 and 1.51 translucent geometry started rendering differently

1.50
https://cesiumjs.org/releases/1.50/Apps/Sandcastle/?src=CZML%20Custom%20Properties.html

image

1.51 and 1.52
https://cesiumjs.org/Cesium/Build/Apps/Sandcastle/?src=CZML%20Custom%20Properties.html&label=All

image

@mramato
Copy link
Contributor Author

mramato commented Dec 3, 2018

@hpinkos @bagnell @lilleyse any ideas here? I haven't bisected yet...

@mramato
Copy link
Contributor Author

mramato commented Dec 3, 2018

This might be specific to this example. For example: http://localhost:8080/Apps/Sandcastle/index.html?src=CZML%20Rectangle.html&label=All seems to work (look at PA)

@hpinkos
Copy link
Contributor

hpinkos commented Dec 3, 2018

Got it, one sec

@hpinkos hpinkos self-assigned this Dec 3, 2018
hpinkos pushed a commit that referenced this issue Dec 3, 2018
@Amber-372
Copy link

but it's still abnormal in version1.52

@hpinkos
Copy link
Contributor

hpinkos commented Dec 4, 2018

@wangxiaolun can you post what you're seeing? It looks correct to me:

image

Note that we recently made some changes that might make it look slightly different than before, but you can still see the changes in lighting for the different faces of the geometry.

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

Successfully merging a pull request may close this issue.

3 participants