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

r73: Report error #7368

Closed
Freeman555 opened this issue Oct 17, 2015 · 15 comments
Closed

r73: Report error #7368

Freeman555 opened this issue Oct 17, 2015 · 15 comments

Comments

@Freeman555
Copy link

Hi there!

I've found some;
->Loader.createMaterial: Unsupported colorAmbient
r_73
working on r72

not r73

@WestLangley
Copy link
Collaborator

Remove

 "colorAmbient": [0.8,0.8,0.8],

from your json file.

@Freeman555
Copy link
Author

@WestLangley works, THX r73 jsfiddle
@mrdoob Three.js Blender Export will be updated without these issue?

@yomotsu
Copy link
Contributor

yomotsu commented Oct 19, 2015

It should be fixed at next rev!
#7374

@Freeman555
Copy link
Author

Cool, so I close it.

@mrdoob
Copy link
Owner

mrdoob commented Oct 19, 2015

I refactored Loader.createMaterial. colorAmbient should have been a warning instead of an error. Will fix!

@SET001
Copy link
Contributor

SET001 commented Nov 1, 2015

If I just remove colorAmbient and also if I will update and use latest blender exporter (from dev) I will end up with mesh with messed colors.

http://stackoverflow.com/questions/33461750/how-does-materials-work-exported-from-json

@mrdoob
Copy link
Owner

mrdoob commented Nov 1, 2015

That's fixed in dev too. Sorry about that.

@SET001
Copy link
Contributor

SET001 commented Nov 1, 2015

@mrdoob but I switched to 74dev and still experiencing this. Maybe I'm doing something wrong?

@mrdoob
Copy link
Owner

mrdoob commented Nov 1, 2015

Could you share the exported model?

@SET001
Copy link
Contributor

SET001 commented Nov 1, 2015

yes, sure - http://pastebin.com/J7pVVRbt
this is a json file from latest blender exporter from dev branch

@yomotsu
Copy link
Contributor

yomotsu commented Nov 1, 2015

I think @SET001 's question is not a bug. It seems your JSON missing colorDiffuse and uvs or VertexColor.

Make sure to check the exporter setting, and this question supposed to be posted Stack Overflow.

@mrdoob
Copy link
Owner

mrdoob commented Nov 2, 2015

@yomotsu thanks!

@SET001
Copy link
Contributor

SET001 commented Nov 2, 2015

@yomotsu thanx )

@mrdoob
Copy link
Owner

mrdoob commented Nov 2, 2015

@yomotsu I'm thinking I should remove support for DbgColor as I an only see it causing confusion like this. What do you think?

@yomotsu
Copy link
Contributor

yomotsu commented Nov 2, 2015

@mrdoob That is fine for me. I don't think it is helpful too...
Actually I don't know the purpose of DbgColor. It is just generated while making materials array, using lookup.

I'm not sure but might be for compatibility with other converters like this

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

5 participants