[Bf-cycles] Squared Samples feature

[Thomas Dinges]

Hi,
thanks for all the feedback.
I commited a improved version of all that now, and it should make some 
things easier to understand.
http://www.pasteall.org/pic/show.php?id=57980

Best regards,
Thomas


Am 22.08.2013 21:35, schrieb Greg Zaal:
> 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.
>
> As for the labeling, the simplest is usually the best. Just #² 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.
>
>
> On 22 August 2013 17:51, Brecht Van Lommel <brechtvanlommel at pandora.be 
> <mailto:brechtvanlommel at pandora.be>> wrote:
>
>     I think this option should definitely work for both progressive and
>     non-progressive mode. I do not think progressive refinement should
>     influence this design decision, using it together with non-progressive
>     mode should work fine, we just have to do some renaming to clear up
>     the confusion.
>
>     I don't really have a suggestion for the label, but would be nice if
>     it was just a single compact line.
>
>     On Thu, Aug 22, 2013 at 5:19 PM, Matthew Heimlich
>     <matt.heimlich at gmail.com <mailto:matt.heimlich at gmail.com>> wrote:
>     > As you know, I like the renaming and label ideas, but don't
>     think it makes
>     > any sense to have square samples enabled for progressive
>     rendering at all.
>     > Exact values for sampling are rarely ever used or needed when
>     progressively
>     > rendering, so it would just be more UI clutter, IMO. There's a
>     difference
>     > between a set of values in non-progressive mode purposely chosen
>     for your
>     > unique scene and materials therein, and a single value that you
>     let run
>     > until things are clean enough.
>     >
>     >
>     > On Thu, Aug 22, 2013 at 10:53 AM, Thomas Dinges
>     <blender at dingto.org <mailto:blender at dingto.org>> wrote:
>     >>
>     >> Hi,
>     >> I discussed the implementation with our artist module members
>     Greg and
>     >> Matt, also got some feedback from Brecht.
>     >> But I am still not sure how to proceed. Suggested changes:
>     >>
>     >> * Re-enable for Progressive integrator again, for consistency.
>     >> * Rename property to "Square Samples" (present tense), so it's
>     a bit
>     >> more clear that the action is not already done.
>     >> * Add a label which displays the total amount of Samples. It
>     would be
>     >> nice to do this per ray type for the Non-Progressive
>     integrator, like:
>     >> Total AA Samples = AA²
>     >> Total Diffuse Samples (Diffuse² * AA²)
>     >>
>     >> but I don't want to waste much space either in the Sampling panel.
>     >>
>     >> Ideas and improvements how this could be tackled are welcome.
>     >>
>     >> Thomas
>     >>
>     >> --
>     >> Thomas Dinges
>     >> Blender Developer, Artist and Musician
>     >>
>     >> www.dingto.org <http://www.dingto.org>
>     >>
>     >> _______________________________________________
>     >> Bf-cycles mailing list
>     >> Bf-cycles at blender.org <mailto:Bf-cycles at blender.org>
>     >> http://lists.blender.org/mailman/listinfo/bf-cycles
>     >
>     >
>     >
>     > _______________________________________________
>     > Bf-cycles mailing list
>     > Bf-cycles at blender.org <mailto:Bf-cycles at blender.org>
>     > http://lists.blender.org/mailman/listinfo/bf-cycles
>     >
>     _______________________________________________
>     Bf-cycles mailing list
>     Bf-cycles at blender.org <mailto:Bf-cycles at blender.org>
>     http://lists.blender.org/mailman/listinfo/bf-cycles
>
>
>
>
> _______________________________________________
> Bf-cycles mailing list
> Bf-cycles at blender.org
> http://lists.blender.org/mailman/listinfo/bf-cycles


-- 
Thomas Dinges
Blender Developer, Artist and Musician

www.dingto.org

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.blender.org/pipermail/bf-cycles/attachments/20130822/fbfde4e1/attachment.htm