Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Python wrapper should return some kind of mapping from integer material ID to string material name #131

@mikeroberts3000

Description

@mikeroberts3000

Hello there,

This obj loader looks great. As far as I can tell though, there there is a subtle design oversight in the Python wrapper.

For each face, the Python wrapper returns an integer material ID. The Python wrapper also returns a dictionary of materials (i.e., a mapping from string material names to various material properties). But there is no mapping from integer material IDs to string material names.

Suppose some face has a material_id of 1. I have no way of knowing which material properties this material_id refers to, because Python dictionaries are unordered and cannot be indexed with the material ID.

Cheers,
Mike

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions