Moo-Ack-Productions / MCprep

Blender python addon to increase workflow for creating minecraft renders and animations
https://theduckcow.com/MCprep
GNU General Public License v3.0
268 stars 25 forks source link

Incorrect argument in `set_cycles_texture` #531

Closed StandingPadAnimations closed 5 months ago

StandingPadAnimations commented 5 months ago

Did the issue happen after restarting Blender?

Check known issues

What happened?

MCprep 3.5.2 has a bug where we use the wrong keyword argument here: https://github.com/Moo-Ack-Productions/MCprep/blob/5aaee32a88cab2ff7dc97037980596600379a899/MCprep_addon/materials/generate.py#L351-L352

use_extra_passes is now extra_passes

What steps are needed to produce the bug?

Use texture swap

What version of MCprep are you using?

MCprep 3.5.2

What version of Blender are you using?

All of them

TheDuckCow commented 5 months ago

It's also the top 1 and 2 spots by quite a margin in the send-to-developer report (just showing the third to show how distant it is, but is not related):

Screen Shot 2024-01-08 at 10 54 16 AM

Full error for future onlookers, thanks for being quick to find a fix:

482, in execute
 res += generate.set_texture_pack(mat, folder, self.useExtraMaps)
 File "<addon_path>/generate.py", line 351, in set_texture_pack
 _ = set_cycles_texture(
 TypeError: set_cycles_texture() got an unexpected keyword argument 'use_extra_passes'
StandingPadAnimations commented 5 months ago

Fixed in #532