-
Notifications
You must be signed in to change notification settings - Fork 19
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
[WIP] major material rework #100
Conversation
27c54ac
to
1cac534
Compare
tests/common/cases/test_utils.py
Outdated
def test_shader_material_type_name_fallback(self): | ||
mesh = get_mesh(shader_mats=True) | ||
|
||
for source in mesh.shader_materials: | ||
source.header.type_name = 'LoremIpsum' | ||
source.properties = [] | ||
|
||
(material, principled) = create_material_from_shader_material( | ||
self, mesh.name(), source) | ||
actual = retrieve_shader_material(self, material, principled) | ||
source.header.type_name = 'DefaultW3D.fx' | ||
compare_shader_materials(self, source, actual) | ||
|
||
def test_shader_material_type_name_upgrade_to_normal_mapped(self): | ||
mesh = get_mesh(shader_mats=True) | ||
|
||
for source in mesh.shader_materials: | ||
source.header.type_name = 'LoremIpsum' | ||
|
||
(material, principled) = create_material_from_shader_material( | ||
self, mesh.name(), source) | ||
actual = retrieve_shader_material(self, material, principled) | ||
source.header.type_name = 'NormalMapped.fx' | ||
compare_shader_materials(self, source, actual) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
readd these
- support prelit materials and multiple material passes | ||
- support for per face materials | ||
- support for per face surface types via face maps | ||
- support for vertex colors | ||
- apply modifiers to meshes on export |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move these to appropriate version
Found a solution for the NodeSocketInterface problem as described here: |
replaced by PR #170 |
Got most of the stuff for import working so far. Export still needs some work.
Encountered some issues with missing functionality in the blender python api in order to complete this:
This way stuff like the alpha pipeline can not be provided as node group templates since a user would have to fill in the default values for math nodes himself which is not feasable.