<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">Hi,<br>
thanks for all the feedback.<br>
I commited a improved version of all that now, and it should make
some things easier to understand.<br>
<a class="moz-txt-link-freetext" href="http://www.pasteall.org/pic/show.php?id=57980">http://www.pasteall.org/pic/show.php?id=57980</a><br>
<br>
Best regards,<br>
Thomas<br>
<br>
<br>
Am 22.08.2013 21:35, schrieb Greg Zaal:<br>
</div>
<blockquote
cite="mid:CAN5-zsPtAmi-EZ_7PLu0C2Ervqnwed4HormeoWAW09F2Nnc8zQ@mail.gmail.com"
type="cite">
<div dir="ltr">It should definitely be added to the Progressive
integrator as well. That's pretty much all I use and it'd be
very welcome. The idea that it shouldn't be added for the
progressive integrator could equally be argued against the
non-progressive too.
<div>
<br>
</div>
<div>As for the labeling, the simplest is usually the best. Just
#<span style="font-family:arial,sans-serif;font-size:13px">²
is fine by me. A label to display anything more is a bit
redundant, since after reading just the tooltip it's clear
what happens and the user does the math themselves to see
how many diffuse samples will be done. Though having never
really used the non-progressive integrator I don't know how
confusing it could get. One or two lines below showing the
totals of each sample type would be just fine anyway.</span></div>
<div class="gmail_extra"><br>
<br>
<div class="gmail_quote">On 22 August 2013 17:51, Brecht Van
Lommel <span dir="ltr"><<a moz-do-not-send="true"
href="mailto:brechtvanlommel@pandora.be" target="_blank">brechtvanlommel@pandora.be</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">I think
this option should definitely work for both progressive
and<br>
non-progressive mode. I do not think progressive
refinement should<br>
influence this design decision, using it together with
non-progressive<br>
mode should work fine, we just have to do some renaming to
clear up<br>
the confusion.<br>
<br>
I don't really have a suggestion for the label, but would
be nice if<br>
it was just a single compact line.<br>
<div class="HOEnZb">
<div class="h5"><br>
On Thu, Aug 22, 2013 at 5:19 PM, Matthew Heimlich<br>
<<a moz-do-not-send="true"
href="mailto:matt.heimlich@gmail.com">matt.heimlich@gmail.com</a>>
wrote:<br>
> As you know, I like the renaming and label ideas,
but don't think it makes<br>
> any sense to have square samples enabled for
progressive rendering at all.<br>
> Exact values for sampling are rarely ever used or
needed when progressively<br>
> rendering, so it would just be more UI clutter,
IMO. There's a difference<br>
> between a set of values in non-progressive mode
purposely chosen for your<br>
> unique scene and materials therein, and a single
value that you let run<br>
> until things are clean enough.<br>
><br>
><br>
> On Thu, Aug 22, 2013 at 10:53 AM, Thomas Dinges
<<a moz-do-not-send="true"
href="mailto:blender@dingto.org">blender@dingto.org</a>>
wrote:<br>
>><br>
>> Hi,<br>
>> I discussed the implementation with our
artist module members Greg and<br>
>> Matt, also got some feedback from Brecht.<br>
>> But I am still not sure how to proceed.
Suggested changes:<br>
>><br>
>> * Re-enable for Progressive integrator again,
for consistency.<br>
>> * Rename property to "Square Samples"
(present tense), so it's a bit<br>
>> more clear that the action is not already
done.<br>
>> * Add a label which displays the total amount
of Samples. It would be<br>
>> nice to do this per ray type for the
Non-Progressive integrator, like:<br>
>> Total AA Samples = AA²<br>
>> Total Diffuse Samples (Diffuse² * AA²)<br>
>><br>
>> but I don't want to waste much space either
in the Sampling panel.<br>
>><br>
>> Ideas and improvements how this could be
tackled are welcome.<br>
>><br>
>> Thomas<br>
>><br>
>> --<br>
>> Thomas Dinges<br>
>> Blender Developer, Artist and Musician<br>
>><br>
>> <a moz-do-not-send="true"
href="http://www.dingto.org" target="_blank">www.dingto.org</a><br>
>><br>
>>
_______________________________________________<br>
>> Bf-cycles mailing list<br>
>> <a moz-do-not-send="true"
href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a><br>
>> <a moz-do-not-send="true"
href="http://lists.blender.org/mailman/listinfo/bf-cycles"
target="_blank">http://lists.blender.org/mailman/listinfo/bf-cycles</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> Bf-cycles mailing list<br>
> <a moz-do-not-send="true"
href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a><br>
> <a moz-do-not-send="true"
href="http://lists.blender.org/mailman/listinfo/bf-cycles"
target="_blank">http://lists.blender.org/mailman/listinfo/bf-cycles</a><br>
><br>
_______________________________________________<br>
Bf-cycles mailing list<br>
<a moz-do-not-send="true"
href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a><br>
<a moz-do-not-send="true"
href="http://lists.blender.org/mailman/listinfo/bf-cycles"
target="_blank">http://lists.blender.org/mailman/listinfo/bf-cycles</a><br>
</div>
</div>
</blockquote>
</div>
<br>
</div>
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
Bf-cycles mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a>
<a class="moz-txt-link-freetext" href="http://lists.blender.org/mailman/listinfo/bf-cycles">http://lists.blender.org/mailman/listinfo/bf-cycles</a>
</pre>
</blockquote>
<br>
<br>
<pre class="moz-signature" cols="72">--
Thomas Dinges
Blender Developer, Artist and Musician
<a class="moz-txt-link-abbreviated" href="http://www.dingto.org">www.dingto.org</a></pre>
</body>
</html>