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

dae -> 3D tiles, missing textures #73

Closed
DFic opened this issue Dec 12, 2018 · 8 comments
Closed

dae -> 3D tiles, missing textures #73

DFic opened this issue Dec 12, 2018 · 8 comments

Comments

@DFic
Copy link

DFic commented Dec 12, 2018

Tried to convert dae model to 3D tiles, and it's working, except the textures are missing in resulting 3D tiles model.

Link to original thread at Cesium forum:
https://cesiumjs.org/forum/#!msg/cesium-dev/2IlV8hvDR3Q/oF6s0KvmCAAJ

@lilleyse
Copy link

Hi @DFic,

If you drag and drop both medvedgrad1a.dae and the medvedgrad1a texture folder onto the Upload Data page you should get a tileset with textures. Maybe only the .dae file was uploaded when you tried?

picture

@DFic
Copy link
Author

DFic commented Dec 13, 2018

Thanks Sean,
drag and drop is a solution for textures, I tried to use "open" for textures but then path was lost.

Just one more thing, noticed that 3D tiles models are bit darker then same glft models ( please see attached comparison image).
glft_3dtiles_light

@lilleyse
Copy link

3D Tiles uses glTF 2.0 which has a different shading model than glTF 1.0 and may account for those differences.

To make the tileset brighter try setting tileset.lightColor. The default is (1.5, 1.4, 1.2)

@DFic
Copy link
Author

DFic commented Dec 14, 2018

Having a hard time finding a detailed reference about tileset.lightColor, especially about those three parameters.
General info only in https://cesiumjs.org/Cesium/Build/Documentation/Cesium3DTileset.html?classFilter=cesium3dtileset#lightColor

pls, could u share a small code example?

@OmarShehata
Copy link

OmarShehata commented Dec 14, 2018

Try:

tileset.lightColor = new Cesium.Cartesian3(3, 2.8, 2.4)

Does that make it brighter for you?

@lilleyse
Copy link

I'm going to close this for now. @DFic let us know if you still have any questions.

@shueseman-skyline
Copy link

This doesn't seem to work, tried in Cesium versions 1.53 and 1.56. I, too, am looking to "brighten" up a tileset. You can test in your sandcastle (https://cesiumjs.org/Cesium/Build/Apps/Sandcastle/?src=3D%20Tiles%20Photogrammetry.html) and add the line above (tileset.lightColor = new Cesium.Cartesian3(3, 2.8, 2.4);) into the sandcastle

@OmarShehata
Copy link

@lilleyse can correct me if I'm wrong here, but I think this is because photogrammetry tilesets use an unlit material so that they are unaffected by light sources in the scene.

If you try this on the 3D Tiles interior model here it works:

https://cesiumjs.org/Cesium/Build/Apps/Sandcastle/?src=3D%20Tiles%20Interior.html

Try:

var lightIntensity = 20;
var sunColor = new Cesium.Cartesian3(3, 2.8, 2.4);

Cesium.Cartesian3.multiplyByScalar(sunColor, lightIntensity, tileset.lightColor) 

I know you can apply styles CesiumGS/cesium#7255 to photogrammetry tilesets but that won't help make it brighter though.

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

4 participants