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

glTF 2.0 model lose light on Andoird #7403

Closed
alx696 opened this issue Dec 11, 2018 · 4 comments
Closed

glTF 2.0 model lose light on Andoird #7403

alx696 opened this issue Dec 11, 2018 · 4 comments

Comments

@alx696
Copy link

alx696 commented Dec 11, 2018

  • Ubuntu 18.04 Chrome 71.0.3578.80
  • Android 8.0 Chrome 70.0.3538.80 Samsung S8
  • Cesium 1.52

Problem:

  1. Aplha lost;
  2. Model dark on Android;
  3. Below is whiter than above;
  4. White Color yellowing(1.43 right, 1.45 little yellowing, 1.52 much yellowing).

Blender file:
https://github.com/alx696/share/raw/master/gltf/box.zip

glTF2.0 file:
https://github.com/alx696/share/raw/master/gltf/box.gltf

Disable shadows of gltf not work:

      let model = viewer.scene.primitives.add(Cesium.Model.fromGltf({
        url: `gltf/box.gltf`,
        modelMatrix: transforms,
        shadows: Cesium.ShadowMode.DISABLED,
        minimumPixelSize: 128,
        maximumScale: 20000
      }));

Android WebGL Report: #6622 (comment)

Model render:
Model render

Ubuntu 18.04 Chrome 71.0.3578.80:
Ubuntu 18.04 Chrome 71.0.3578.80

Android 8.0 Chrome 70.0.3538.80 Samsung S8:
Android 8.0 Chrome 70.0.3538.80 Samsung S8

Just now, i found a old phone(GPU Mali-T720). The model show well. But new phones are dark.
old

Old Phone WebGL Report:

@likangning93
Copy link
Contributor

I'm actually able to get the same effect on Adreno 308:

screenshot_20181211-160617

@alx696 thanks for the model, for anyone else testing here's the scene on cesiumjs.

@alx696 alx696 changed the title glTF 2.0 models dark on Andoird glTF 2.0 model lose light on Andoird Dec 14, 2018
@OmarShehata
Copy link
Contributor

This should be fixed with #7944, and will be out with CesiumJS 1.59 coming out next week.

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

No branches or pull requests

3 participants