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

VRML -> GLTF : Black faces #1046

Closed
nbouayad opened this issue Jul 20, 2017 · 3 comments
Closed

VRML -> GLTF : Black faces #1046

nbouayad opened this issue Jul 20, 2017 · 3 comments

Comments

@nbouayad
Copy link

nbouayad commented Jul 20, 2017

Hello,
I am writing a VRML to GLTF/GLB conveter.
I am advancing well but I think that I am having problems with normals.I get black faces (I recompute the normals using the normalized sum of faces normals, also tried weighted areas, or weighed angles, but same result).
Even if materials do not match (That is another problem...), black faces look like a bug in my processing (I believe)
Please see the files below (Note that the GLTF mesh is a merge of the VRML shapes) :

sample.zip

sample.wrl : The original VRML
sample.bin + sample.gltf : The resulting GLTF
sample-vrml-bottom.PNG : Screenshot of the bottom for VRML
sample-vrml-top.PNG : Screenshot of the top for VRML
sample-glt-bottom.PNG : Screenshot of the bottom for GLTF
sample-glt-top.PNG Screenshot of the top for GLTF

Sorry if It looks dumb, but I am a pragmatic developer, not a 3d expert :)
Any help will be appreciated,I am just stuck.

@nbouayad nbouayad changed the title Black faces VRML -> GLTF : Black faces Jul 20, 2017
@donmccurdy
Copy link
Contributor

donmccurdy commented Jul 22, 2017

I'm not sure how to help you debug your converter, sorry, but FYI the bufferView uri needs to be changed to sample.bin. With that change you can preview the model in https://gltf-viewer.donmccurdy.com/.

Here's a version of the model with that corrected:

sample 2.zip

screen shot 2017-07-21 at 5 21 18 pm

@pjcozzi
Copy link
Member

pjcozzi commented Dec 24, 2017

I don't think there are any other actions here.

@nbouayad please keep us posted on your VRML to glTF converter. We would be happy to list it here: https://github.com/KhronosGroup/glTF#gltf-tools

@pjcozzi pjcozzi closed this as completed Dec 24, 2017
@donmccurdy
Copy link
Contributor

@nbouayad were you able to get your converter working?

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