Default values for materials[REOPEN]

[lucian-student]

Default variables for material Get method

Allow setting default variable, instead of using default hardcoded variablles.

Hardcoded variables feel very limiting, There should be way to override them.

Solution

• Maybe overload Get method, with extra parameter with default.
• Boolean flag to enable defaults.
• add to aiReturn param missing option

Currently API seems to not support such option, which seems like oversight.

[tellypresence]

@lucian-student
In the past I remember thinking that there were some compile/build-time #defines that I felt would have been better if they were converted to variables that could be set at runtime. I think that's what you're referring to? But unfortunately without any specifics, it's difficult to understand your request.

Would suggest if you can at the very least provide names of specific compile-time constants that you'd like converted to runtime variables (and preferably provide links to source files where they're defined and used) you might have better luck starting a conversation

(Also, title says [REOPEN] but it's a mystery what the original issue was, please provide a link to the closed issue you're referring to so it's clear)

[lucian-student]

@tellypresence thank u for answering my request. Iam refering to material loading and default variables. File with hardcoded variables inside constructor.

Currently when calling:

aiColor3d color = (0.f,0.f,0.f);
material->Get(AI_MATKEY,color);

U can't override default hardcoded variable. So i propose to allow such possibility, when material attribute isn't present in file.

Using default variables inside constructor without option to override them doesn't seem to be appropriate in this case.