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

Feature request: Make material import optional #131

Open
caniko opened this issue Mar 11, 2021 · 3 comments
Open

Feature request: Make material import optional #131

caniko opened this issue Mar 11, 2021 · 3 comments

Comments

@caniko
Copy link

caniko commented Mar 11, 2021

As mentioned in #53, material files may not have the same name. Moreover, in my case, I don't have any .mtl files paired to my obj. files. I just want to use the parser to import vertexes, normals, and faces to my script.

Solution:
Make a keyword argument that takes file path for .mtl files. Also add a secondary keyword argument that is a bool for skipping material import. Use a default material (grey?) for rendering if material skip is set to True.

@caniko caniko changed the title Make material import optional Feature request: Make material import optional Mar 11, 2021
@einarf
Copy link
Member

einarf commented Mar 11, 2021

Do you have a real life example of materials names and usemtl statements?

@einarf
Copy link
Member

einarf commented Mar 11, 2021

If you just want to use the parser to import vertexes, normals, and faces you can use the create_materials=True parameter so everything will use a default materal instead? (assuming you have no mtl files)

@caniko
Copy link
Author

caniko commented Mar 15, 2021

Do you have a real life example of materials names and usemtl statements?

No, but it seems like a kwarg worth having for the comfort of the enduser.

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

2 participants