<div dir="ltr">Hi,<div><br></div><div>Thanks for sharing the work!</div><div><br></div><div>As for getting commit access to our git, think it&#39;s all reasonable to move development of such feature under <a href="http://blender.org">blender.org</a> infrastructure (in a branch initially, so we don&#39;t ruing master branch immediately). For that you&#39;ll need to be registered at <a href="http://developer.blender.org">developer.blender.org</a> first.</div><div><br></div><div>Also, if you can drop by to IRC #blendercouders ar <a href="http://irc.freenode.net">irc.freenode.net</a> it will speed up some communication. You can poke me in there (nick hackerman), or just ask the guys around there if you&#39;ll have some difficulties or such.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 17, 2016 at 9:09 AM, &quot;Pascal Schön&quot; <span dir="ltr">&lt;<a href="mailto:pascal_schoen@gmx.net" target="_blank">pascal_schoen@gmx.net</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div style="font-family:Verdana;font-size:12.0px"><div>I have pushed my implementation in a bitbucket repository with the branch name cycles_disney_brdf. You can clone (or add it as a new remote) and test it, if you want:</div>

<div> </div>

<div><a href="https://bitbucket.org/VanCantus/blender" target="_blank">https://bitbucket.org/VanCantus/blender</a></div>

<div> </div>

<div>But now I have another question for committing to the original Blender repo. I need commit rights but don&#39;t know how to get it. I&#39;ve read that I have to get in contact with one of the admins, but on the wiki I&#39;m always refered to the mailing lists. Thus I just wanted to ask, if I only have to write to the bf-committers list or what do I have to do?</div>

<div> </div>

<div>You can answer directly to my mail address if is refered to my committing question, because it doesn&#39;t have to be discussed here I think. ;-)</div>

<div> </div>

<div>Pascal</div>

<div> 
<div name="quote" style="margin:10px 5px 5px 10px;padding:10px 0 10px 10px;border-left:2px solid #c3d9e5;word-wrap:break-word">
<div style="margin:0 0 10px 0"><b>Gesendet:</b> Donnerstag, 12. Mai 2016 um 13:54 Uhr<br>
<b>Von:</b> &quot;Pascal Schön&quot; &lt;<a href="mailto:pascal_schoen@gmx.net" target="_blank">pascal_schoen@gmx.net</a>&gt;<br>
<b>An:</b> &quot;Discussion list to assist Cycles render engine developers&quot; &lt;<a href="mailto:bf-cycles@blender.org" target="_blank">bf-cycles@blender.org</a>&gt;<br>
<b>Betreff:</b> Re: [Bf-cycles] Disney BRDF Implementation in Cycles</div><div><div class="h5">

<div name="quoted-content">
<div style="background-color:rgb(255,255,255)">
<div style="line-height:1;padding:0.5em">Yes, that&#39;s excacly what it is. It&#39;s an approximation of the glossiness of a clear varnish. It&#39;s described in the paper that Disney published and I refered to in my first mail.<br>
<br>
Pascal</div>

<div style="line-height:1;padding:0.3em">On 5/11/16, 19:39 Zauber Paracelsus &lt;<a href="mailto:zauber@gridmail.org" target="_blank">zauber@gridmail.org</a>&gt; wrote:
<blockquote class="gmail_quote" style="margin:0.8ex 0.0pt 0.0pt 0.8ex;border-left:1.0px solid rgb(204,204,204);padding-left:1.0ex">
<div>Pardon my asking, but what is clearcloat?  Is that a simulation of having a thin layer of glossy/transmissive material on top of another?  (eg: simulating things like varnish)<br>
<br>
On 05/11/2016 01:36 PM, &quot;Pascal Schön&quot; wrote:</div>

<blockquote>
<div style="line-height:1;padding:0.5em">Thanks for your quick replies.<br>
<br>
Some words to the implementation:<br>
<br>
I&#39;ve already splitted the BRDF into three closures after trying to get a single closure implementation to work correctly. But it came to my mind that this would be against the cycles architecture of splitting stuff into diffuse and glossy. And it made it also much more complex.<br>
<br>
The three closures are:<br>
- disney_diffuse(...): Calculates the Burley diffuse term, subsurface scattering and the sheen<br>
- disney_specular(...): Calculates the &quot;normal&quot; specular highlight and the metalness (also anisotropy is included here)<br>
- disney_clearcoar(...): By this the clearcoat specular highlight is calculated (less bright and no anisotropy)<br>
<br>
In an OSL script, which is also the script for the node, these three closures are called correctly and added together.<br>
<br>
I&#39;ve already seen the updated version of the Disney BRDF but I wanted to focus on the more simple one that was proposed in 2012. The additionals (like getting from a BRDF to a BSDF) would be one of the next steps.<br>
<br>
I&#39;m going to commit my implementation to a new branch next week, because this week is a bit stressful and as it is my first commit, I wanted to take some time for it.<br>
<br>
Pascal</div>

<div style="line-height:1;padding:0.3em">On 11.05.16, 19:06 Stefan Werner<a href="http://swerner@smithmicro.com" target="_blank"> &lt;swerner@smithmicro.com&gt;</a> wrote:

<blockquote class="gmail_quote" style="margin:0.8ex 0.0pt 0.0pt 0.8ex;border-left:1.0px solid rgb(204,204,204);padding-left:1.0ex">Eventually, it might be interesting to split it into separate closures, so that one can still use separate ray counts for specular/diffuse/transmission and get them to render into separate passes. This will probably require to put fresnel back into the closures (going back in history, it looks like Cycles initially had that).<br>
<br>
Stefan Werner | Senior Software Engineer; Productivity &amp; Graphics<br>
Smith Micro Software, Inc. |<a href="http://www.smithmicro.com" target="_blank"> http://www.smithmicro.com</a> &lt;<a href="http://www.smithmicro.com/" target="_blank">http://www.smithmicro.com/</a>&gt;<br>
<br>
<br>
<br>
<br>
<br>
On 5/11/16, 4:18 PM,<a href="http://bf-cycles-bounces@blender.orgonbehalfofLukasStockner" target="_blank"> &quot;bf-cycles-bounces@blender.org on behalf of Lukas Stockner&quot;</a><a href="http://bf-cycles-bounces@blender.orgonbehalfoflukas.stockner@freenet.de" target="_blank"> &lt;bf-cycles-bounces@blender.org on behalf of lukas.stockner@freenet.de&gt;</a> wrote:<br>
<br>
&gt;Hi!<br>
&gt;<br>
&gt;The Disney BRDF shader is actually pretty interesting for Cycles!<br>
&gt;I&#39;ve played around with it for a while as well, but my code isn&#39;t fully<br>
&gt;working yet, so it&#39;d be awesome to have your code around.<br>
&gt;<br>
&gt;Some notes:<br>
&gt;- There&#39;s actually a later revision of the paper you linked with a few<br>
&gt;more details, might be handy:<br>
&gt;<a href="https://disney-animation.s3.amazonaws.com/uploads/production/publication_asset/48/asset/s2012_pbs_disney_brdf_notes_v3.pdf" target="_blank">https://disney-animation.s3.amazonaws.com/uploads/production/publication_asset/48/asset/s2012_pbs_disney_brdf_notes_v3.pdf</a><br>
&gt;- There&#39;s also a completely revised BSDF that includes actual subsurface<br>
&gt;scattering and transmission, and they note that it solves a few issues<br>
&gt;with energy conservation and path tracing in general that the original<br>
&gt;2012 BRDF has:<br>
&gt;<a href="http://blog.selfshadow.com/publications/s2015-shading-course/burley/s2015_pbs_disney_bsdf_notes.pdf" target="_blank">http://blog.selfshadow.com/publications/s2015-shading-course/burley/s2015_pbs_disney_bsdf_notes.pdf</a><br>
&gt;- I&#39;m currently working on energy-conserving microfacet closures, which<br>
&gt;would pretty much solve the issue that caused them to include the sheen<br>
&gt;stuff - at the cost of not being 100% compatible to other implementations.<br>
&gt;<br>
&gt;But that can all be handled later - in general, it would be awesome to<br>
&gt;have the Disney BRDF/BSDF in Cycles, and your code would certainly be<br>
&gt;welcome! Even if we go with the revised BSDF, it is pretty similar to<br>
&gt;the original BRDF, so it could be upgraded easily.<br>
&gt;<br>
&gt;Lukas<br>
&gt;<br>
&gt;<br>
&gt;Am 11.05.2016 um 15:57 schrieb &quot;Pascal Schön&quot;:<br>
&gt;&gt; Hey guys,<br>
&gt;&gt;<br>
&gt;&gt; For a few months now I’m implementing (on a private repository) the<br>
&gt;&gt; Disney BRDF shader<br>
&gt;&gt; (<a href="https://disney-animation.s3.amazonaws.com/library/s2012_pbs_disney_brdf_notes_v2.pdf" target="_blank">https://disney-animation.s3.amazonaws.com/library/s2012_pbs_disney_brdf_notes_v2.pdf</a>)<br>
&gt;&gt; as a new node in Cycles. By now there have only been a few external<br>
&gt;&gt; approaches, where a render server (like in Appleseed or Renderman) is<br>
&gt;&gt; accessed by a Blender plugin. But it would be nice to get access to this<br>
&gt;&gt; without any additional software to install.<br>
&gt;&gt;<br>
&gt;&gt; The current state of my implementation is, that the OSL closure is<br>
&gt;&gt; finished and has also been tested against Renderman. Both acted the same<br>
&gt;&gt; in my testcases (except for the rendering approches by the two<br>
&gt;&gt; renderers, which cause some minor differences in indirect lighting).<br>
&gt;&gt; Another step that I’ve taken was to implement the node to easily access<br>
&gt;&gt; the Disney BRDF.<br>
&gt;&gt;<br>
&gt;&gt; Currently OSL has to be activated, because I haven’t done all steps to<br>
&gt;&gt; include the Disney BRDF completely in Cycles. But I think that’s only<br>
&gt;&gt; one small step to take. There I hope to get some help by the community<br>
&gt;&gt; because it&#39;s a bit confusing where to add which code fragment.<br>
&gt;&gt;<br>
&gt;&gt; You can see some screenshots here:<br>
&gt;&gt;<a href="https://www.dropbox.com/sh/j7w4v0bu9a5jhvz/AADSF3nm3LZCZnHgyPMwhrT6a?dl=0" target="_blank"> https://www.dropbox.com/sh/j7w4v0bu9a5jhvz/AADSF3nm3LZCZnHgyPMwhrT6a?dl=0</a><br>
&gt;&gt;<br>
&gt;&gt; I would be pleased if you tell me if this would be nice to have or if it<br>
&gt;&gt; is too special for Blender.<br>
&gt;&gt;<br>
&gt;&gt; Pascal<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; _______________________________________________<br>
&gt;&gt; Bf-cycles mailing list<br>
&gt;&gt;<a href="http://Bf-cycles@blender.org" target="_blank"> Bf-cycles@blender.org</a><br>
&gt;&gt;<a href="https://lists.blender.org/mailman/listinfo/bf-cycles" target="_blank"> https://lists.blender.org/mailman/listinfo/bf-cycles</a><br>
&gt;&gt;<br>
&gt;<br>
_______________________________________________<br>
Bf-cycles mailing list<br>
<a href="http://Bf-cycles@blender.org" target="_blank">Bf-cycles@blender.org</a><br>
<a href="https://lists.blender.org/mailman/listinfo/bf-cycles" target="_blank">https://lists.blender.org/mailman/listinfo/bf-cycles</a></blockquote>
</div>
 

<fieldset> </fieldset>
 

<pre>_______________________________________________
Bf-cycles mailing list<a href="http://Bf-cycles@blender.org" target="_blank">Bf-cycles@blender.org</a><a href="https://lists.blender.org/mailman/listinfo/bf-cycles" target="_blank">https://lists.blender.org/mailman/listinfo/bf-cycles</a></pre>
</blockquote>
<br>
_______________________________________________ Bf-cycles mailing list <a href="mailto:Bf-cycles@blender.org" target="_blank">Bf-cycles@blender.org</a><a href="https://lists.blender.org/mailman/listinfo/bf-cycles" target="_blank"> https://lists.blender.org/mailman/listinfo/bf-cycles</a></blockquote>
</div>
_______________________________________________ Bf-cycles mailing list <a href="mailto:Bf-cycles@blender.org" target="_blank">Bf-cycles@blender.org</a> <a href="https://lists.blender.org/mailman/listinfo/bf-cycles" target="_blank">https://lists.blender.org/mailman/listinfo/bf-cycles</a></div>
</div>
</div></div></div>
</div></div></div>
<br>_______________________________________________<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/mailman/listinfo/bf-cycles</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div><span style="color:rgb(102,102,102)">With best regards, Sergey Sharybin</span></div></div>
</div>