search

gregzaal / Matalogue

List of node trees to switch between quickly
151 stars 12 forks source link

Added suport for Group Instances #10

Closed JoseConseco closed 7 years ago

JoseConseco commented 7 years ago

Added support for Group Instances = when selecting group instance object (empty with duplication set to group), metalogue will now show materials for objects that are inside this group. Previously it would show 'no material' message. 2017-03-25_18-51-18

gregzaal commented 7 years ago

There are a lot of style changes in your diff like = -> =, whitespace changes, line splitting, etc. that make it hard to review this. Your variable naming style is also inconsistent with PEP8 and the rest of the code.

Otherwise it seems like a good idea to merge this :)

JoseConseco commented 7 years ago

Yep, my IDE (pycharm) did automatic code refactor, that is why so many style changes were introduced. I will try to revert those, and look into pep8 thing.

gregzaal commented 7 years ago

Sorry for the delay, there was still a crapton of style changes to clean up, and a new bug introduced.