<div dir="ltr">No further comments from me, I think the proposal is complete enough.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 15, 2018 at 2:02 PM, Leonardo E. Segovia <span dir="ltr"><<a href="mailto:leonardo.segovia@cs.uns.edu.ar" target="_blank">leonardo.segovia@cs.uns.edu.ar</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
I've updated the proposal with the requested changes. Please read it,<br>
and comment with any questions you may have.<br>
<br>
Thanks in advance.<br>
<div class="HOEnZb"><div class="h5"><br>
2018-03-13 18:51 GMT-03:00 Leonardo E. Segovia <<a href="mailto:leonardo.segovia@cs.uns.edu.ar">leonardo.segovia@cs.uns.edu.<wbr>ar</a>>:<br>
> Hi again,<br>
><br>
> (I'll be replying in different emails so as to quote them)<br>
><br>
> 2018-03-13 13:35 GMT-03:00 Lukas Stockner <<a href="mailto:lukas.stockner@freenet.de">lukas.stockner@freenet.de</a>>:<br>
>> One thing that I noticed reading the proposal: You don't actually need to<br>
>> implement the shader itself twice (in SVM and OSL). Instead, you implement<br>
>> the BSDF itself (sampling and evaluation functions) once, and then you add<br>
>> code in SVM and OSL that creates an instance of that Closure (which<br>
>> typically is a lot easier than actually re-implementing it).<br>
><br>
> Same as with Mr. Brent, thanks for the tips.<br>
><br>
>> Also, a while ago I started looking into implementing the shader - it's<br>
>> extremely WIP, messy and somewhat broken, but it might be helpful for you as<br>
>> a reference as it contains all the code required to implement a new<br>
>> node/closure and a straightforward port of the PBRT code into Cycles. I<br>
>> probably won't continue working on this anytime soon (and certainly not if a<br>
>> GSoC project is also doing the same thing of course), but you can find the<br>
>> current version here:<br>
>> <a href="https://github.com/lukasstockner/blender/tree/hairshader" rel="noreferrer" target="_blank">https://github.com/<wbr>lukasstockner/blender/tree/<wbr>hairshader</a><br>
><br>
> Thanks for your work! I'll make sure to add it to the proposal.<br>
><br>
> --<br>
> Lic. Leonardo E. Segovia<br>
> Departamento de Ciencias e Ingeniería de la Computación<br>
> Universidad Nacional del Sur<br>
> San Andrés 800 - Campus Palihue, B8000 Bahía Blanca, Argentina<br>
<br>
<br>
<br>
--<br>
Lic. Leonardo E. Segovia<br>
Departamento de Ciencias e Ingeniería de la Computación<br>
Universidad Nacional del Sur<br>
San Andrés 800 - Campus Palihue, B8000 Bahía Blanca, Argentina<br>
______________________________<wbr>_________________<br>
Bf-cycles mailing list<br>
<a href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a><br>
<a href="https://lists.blender.org/mailman/listinfo/bf-cycles" rel="noreferrer" target="_blank">https://lists.blender.org/<wbr>mailman/listinfo/bf-cycles</a><br>
</div></div></blockquote></div><br></div>