[Bf-cycles] Squared Samples feature

[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>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> 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>
> 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
> >>
> >> _______________________________________________
> >> Bf-cycles mailing list
> >> 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
> >
> _______________________________________________
> Bf-cycles mailing list
> Bf-cycles at blender.org
> http://lists.blender.org/mailman/listinfo/bf-cycles
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.blender.org/pipermail/bf-cycles/attachments/20130822/76b845a5/attachment.htm