[Bf-cycles] Cycles per BSDF Bump/Normal mapping support

Thomas Dinges blender at dingto.org
Tue Jun 5 09:48:11 CEST 2012


Hi Agus,
thanks for the patch. One thing that is obsolete in your patch is the 
declaration of intensity, bump and NodeTexBase in DNA / RNA. Sockets 
don't need to be exposed there, this can be removed. In fact you don't 
have to add this Node to DNA at all. :) Only thing which is needed is 
the declaration of the node in rna_nodetree_types.h.
I attached a new patch to the patch tracker which removes those things. :)

Regards,
Thomas

Am 05.06.2012 09:01, schrieb Agustin Benavidez:
> Hi Brecht, I think i just finish this patch (only per BSDF bump 
> mapping), I follow your recommendations so i think is ready for review.
> how it looks like in the ui:
> http://www.pasteall.org/pic/show.php?id=32728
>
> I uploaded it in the patch tracker:
> http://projects.blender.org/tracker/index.php?func=detail&aid=31716&group_id=9&atid=127 
> <http://projects.blender.org/tracker/index.php?func=detail&aid=31716&group_id=9&atid=127>
>
> What is not included is normal mapping, I postponed it for the next 
> weekend.
>
> I also optimized several part of the added code, so i think the very 
> little performance lost in the GPU tests have been 
> reduced considerably. for CPU there is not any noticeable performance 
> impact from my tests.
>
> Thank you Brecht
> Agus,
-- 
Thomas Dinges
Blender Developer, Artist and Musician

www.dingto.org



More information about the Bf-cycles mailing list