Support for multiple materials per object/group about tinyobjloader HOT 9 CLOSED

Cool!

I didn't know that .obj can support multiple materials per group.

Let me investigate your patch and think new data structure to support multi-materials per group. Usually the answer is assign per-face material ID like your patch.

from tinyobjloader.

I seems better to start adding support of multi-materials with your patch.
Could you please PR so that I can merge your modification?

from tinyobjloader.

I am sorry but I do not know what PR stands for. If you asking for permission to merge my patch then please go ahead, I have no objections. Just should warn you I modified only tinyobjloader.h/tinyobjloader.cpp and changed interface substantially. So it would be required to fix test/demo programs.

from tinyobjloader.

PR = pull request

https://help.github.com/articles/creating-a-pull-request

PR makes merging process easy.

from tinyobjloader.

Done.

from tinyobjloader.

Thanks!

I'm adding material_ids parameter per mesh, which holds per-face material ID.
This data structure is common for various renderers.

cbba0a8

If you think this modification looks OK, I'll proceed to merge it in master.

from tinyobjloader.

Well everything seems good. It is just a different use case then mine, though I see no significant problems adapting it. Unfortunately I can not test it. If everything works from your side, you probably should proceed with merge. Was happy to contribute:).

from tinyobjloader.

At least it seems working for cryteck sponza .obj > Unfortunately I can not test it

from tinyobjloader.

Merged into master. Thanks!

from tinyobjloader.