search

CesiumGS / obj2gltf

Convert OBJ assets to glTF
Apache License 2.0
1.71k stars 307 forks source link

why my image can't loaded?like this,my code,actually display and expected display all show in the blocks #50

Closed Crise420WMT closed 7 years ago

Crise420WMT commented 7 years ago

display code origin

lilleyse commented 7 years ago

Are you able to send the model over?

You can use try running on the major-cleanup branch that is currently in PR https://github.com/AnalyticalGraphicsInc/obj2gltf/pull/49. It's possible it has been fixed in that branch.

Crise420WMT commented 7 years ago

image image i clone from your tree,then npm install,,,and node bin/obj2gltf.js -i xxx -o xxx,then appear error,and are you sure is correct?

Crise420WMT commented 7 years ago

i find,when a plane more than 4 vertices,it can't convert,,,did your major clipping-up branch resolved?

lilleyse commented 7 years ago

From above it seems like the npm install may have failed.

Also if you have any problematic models you can upload them to github for me to check.

Crise420WMT commented 7 years ago

thanks....undering files are the converted gltf file's showing image and the obj file,then you will find,there are some problem..... Bar Stool 20.zip toolbar

lilleyse commented 7 years ago

Thanks for sending it over.

I've seen this problem with https://github.com/AnalyticalGraphicsInc/obj2gltf/issues/33 as well. Right now the code does not support faces that have more than 4 sides.

I'm reopening the issue since it seems like an important thing to consider.

Crise420WMT commented 7 years ago

maybe undering link will help you....thanks,waiting for your success support more than three vertices in one face [fix this problem] (Fix OBJLoader to support more than three vertices in one face by yqrashawn · Pull Request #10917 · mrdoob/three.js · GitHub)

Crise420WMT commented 7 years ago

maybe undering link will help you....thanks,waiting for your success more than 3 vertices

threejs author's answer

Crise420WMT commented 7 years ago

it is closed? due to my incorrect operation,,, ou my God...how to make it open?

lilleyse commented 7 years ago

Another thing I noticed about this obj is that the mttllib line looks like:

mtllib Bar Stool 20.mtl

The obj format expects a format like:

mtllib filename1 filename2

So the converter treats each word "Bar", "Stool", and "20.mtl" as separate files to look for. It shouldn't be too hard to detect this case. But this explains why the materials and images aren't working.

Crise420WMT commented 7 years ago

are you sure another model in correctly obj format has no problem like this?.... the issue about more than 4 vertices is going on,,,,,,i think it almost resolved,but we i devtools,it errors

lilleyse commented 7 years ago

I'm closing this for now and linking to it to a new issue: https://github.com/AnalyticalGraphicsInc/obj2gltf/issues/55

In the meantime you can use import and export the model from Blender which will turn all faces to triangles or quads.

Also consider fixing the mtl path above: https://github.com/AnalyticalGraphicsInc/obj2gltf/issues/50#issuecomment-288734837