WaveFrontReader MTL support for xyz color-space values

Open
#229 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
45/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
cpp

Research direction

Start by locating the WaveFrontReader MTL parsing entry point and its current RGB color handling. Compare the parser behavior with the linked MTL specification, then cover the listed XYZ, transmission, spectral, and dissolve options as described. Done means the supported XYZ values work and the unsupported options are indicated or documented as requested.

Written by the indexing model from the issue text.

Description

utility

The MTL reader currently only supports RGB values for the colors. There is a variant which writes the colors in the CIE XYZ color-space instead.

Ka r g b
Ka xyz x y z

Kd r g b
Kd xyz x y z

Ks r g b
Ks xyz x y z

There is also a "Transmission Factor" color I don't handle, which I should at least indicate in a debug build.

Tf r g b
Tf xyz x y z

And a Spectral Curve File (.rfl) color option which I don't need or support, but should also document:

Ka spectral file.rtl
Ka spectral file.rtl factor

Kd spectral file.rtl
Kd spectral file.rtl factor

Ks spectral file.rtl
Ks spectral file.rtl factor

Tf spectral file.rtl
Tf spectral file.rtl factor

And finally I note that the 'dissolve' (i.e. alpha) has an optional switch -halo which I don't currently handle.

d factor
d -halo factor

See MTL OBJ materials file.

Dominant language
C++
Stars
860
Forks
169
Avg merge
9h 49m
Merged PRs (30d)
4

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from microsoft/DirectXMesh

All issues in microsoft/DirectXMesh

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.