glTF model appears with black shades #5835
Labels
Comments
|
Could you send over the model you're testing? |
|
Hello,
sure. I am sending it in this mail.
Thank You.
Regards,
Michal
Mocsa4_simplified_3d_mesh.gltf
<https://drive.google.com/file/d/0By2tH5pP-77Jd3UxN3JDYkhfQ00/view?usp=drive_web>
2017-09-18 15:18 GMT+02:00 Sean Lilley <[email protected]>:
… Could you send over the model you're testing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#5835 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ALJYicrCv1n9bAfsM0eVzsYt7ZECXCFMks5sjm21gaJpZM4PanBA>
.
|
|
Thanks for sending the model over. The problem is we are not generating PBR shaders correctly for models that don't have normals. I opened an issue here: #5838. In the meantime you can use the |
|
I also opened up this PR in obj2gltf to prevent transparency of 0, which is another problem I noticed with the glTF: CesiumGS/obj2gltf#107 |
|
Thank You for the advice, but I am afraid it does not work good in cesium
with materialsCommon...
2017-09-19 17:34 GMT+02:00 Sean Lilley <[email protected]>:
… Thanks for sending the model over. The problem is we are not generating
PBR shaders correctly for models that don't have normals. I opened an issue
here: #5838 <#5838>.
In the meantime you can use the --materialsCommon flag with obj2gltf and
this should look correct in Cesium.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#5835 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ALJYifVTQ32LMkqSqQRky3pXNwnQtdHtks5sj9z9gaJpZM4PanBA>
.
|
|
Hm.. would you mind also sending the obj model over? |
|
Ok I realized what the problem was in Let me know how it goes. |
|
Thanks.
And should it work now with --materialsCommon ? Because it doesn't...
2017-09-21 16:50 GMT+02:00 Sean Lilley <[email protected]>:
… Ok I realized what the problem was in obj2gltf and opened a PR. Make sure
to still use the --materialsCommon flag.
CesiumGS/obj2gltf#108
<CesiumGS/obj2gltf#108>.
Let me know how it goes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#5835 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ALJYiR3pDKQGH8HDI-EX7IeusGB1I_8Iks5sknfRgaJpZM4PanBA>
.
|
|
Until the PR is merged make sure you're on the
|
|
It was just merged so being on master is fine too now. |
|
thank you. I am gonna try it right now and I will let you know.
2017-09-21 16:58 GMT+02:00 Sean Lilley <[email protected]>:
… It was just merged so being on master is fine too now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#5835 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ALJYiT0vFFnGHXbumwEdgN_4-0d_7kH6ks5sknmKgaJpZM4PanBA>
.
|
|
I think there there is some issue with the latest master.
[image: Vnorený obrázok 1]
2017-09-21 17:01 GMT+02:00 Michael Knave <[email protected]>:
… thank you. I am gonna try it right now and I will let you know.
2017-09-21 16:58 GMT+02:00 Sean Lilley ***@***.***>:
> It was just merged so being on master is fine too now.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <#5835 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ALJYiT0vFFnGHXbumwEdgN_4-0d_7kH6ks5sknmKgaJpZM4PanBA>
> .
>
|
|
@michaelknave You may need to use the github interface to attach an image. |
|
|
|
Try running |
|
problem has been fixed. But model is still black...even I followed your
recommendations
2017-09-21 18:02 GMT+02:00 Sean Lilley <[email protected]>:
… Try running npm run build.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5835 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ALJYiYzy7DtEWL61avwegA3ALppWfdmNks5skoh2gaJpZM4PanBA>
.
|
|
In sandcastle it doesn't apper lighter
2017-09-21 18:34 GMT+02:00 Michael Knave <[email protected]>:
… problem has been fixed. But model is still black...even I followed your
recommendations
2017-09-21 18:02 GMT+02:00 Sean Lilley ***@***.***>:
> Try running npm run build.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#5835 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ALJYiYzy7DtEWL61avwegA3ALppWfdmNks5skoh2gaJpZM4PanBA>
> .
>
|
|
I'm not sure what else could be the problem. Make sure you are running the latest |
|
it is working finally ! Thanks for help
2017-09-21 18:56 GMT+02:00 Sean Lilley <[email protected]>:
… I'm not sure what else could be the problem. Make sure you are running the
latest obj2gltf. This is the Sandcastle code I'm using:
var viewer = new Cesium.Viewer('cesiumContainer');
function createModel(url, height) {
viewer.entities.removeAll();
var position = Cesium.Cartesian3.fromDegrees(-123.0744619, 44.0503706, height);
var entity = viewer.entities.add({
name : url,
position : position,
model : {
uri : url
}
});
viewer.trackedEntity = entity;
}
createModel('../../SampleData/test/mocsa4/Mocsa4_simplified_3d_mesh.gltf', 5000.0);
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5835 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ALJYiT6Hy4X_IpgxP11JDwh3WcT6qkaGks5skpVTgaJpZM4PanBA>
.
|
|
Good to hear! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have glTF model of the mesh. I have used obj2gltf for conversion. After I loaded it to Cesium, it showed in black colors.
I have tried to open it in different glTF viewers, it showed in correct colors.
I assume there is some bug in cesium.
The text was updated successfully, but these errors were encountered: