References to AA Samples #76340
Labels
No Label
Meta
Good First Issue
Module
Animation & Rigging
Module
Core
Module
Development Management
Module
Eevee & Viewport
Module
Grease Pencil
Module
Modeling
Module
Nodes & Physics
Module
Pipeline, Assets & IO
Module
Platforms, Builds, Tests & Devices
Module
Python API
Module
Rendering & Cycles
Module
Sculpt, Paint & Texture
Module
User Interface
Module
VFX & Video
Priority
High
Priority
Low
Priority
Normal
Status
Archived
Status
Confirmed
Status
Duplicate
Status
Needs Information from Developers
Status
Needs Information from User
Status
Needs Triage
Status
Resolved
Type
Bug
Type
Design
Type
Known Issue
Type
Patch
Type
Report
Type
To Do
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: blender/blender-manual#76340
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
https://docs.blender.org/manual/en/dev/render/cycles/render_settings/sampling.html
There are multiple references to AA samples but no definition of what that value is or where it's set. I guess the samples setting used to be called anti-aliasing samples but that ended in 2.80. Even the new adaptive sampling settings refer to AA samples, yet there's nothing in the current Blender called that. The tooltips in the adaptive sampling panel refers incorrectly to the non-existent AA samples instead of simply samples. (should that be a separate bug since it's not documentation?) It should also be noted that the Python code for this is correctly called cycles.samples.
Added subscriber: @Russ1642
Added subscriber: @Alaska
There are still parts of Cycles that directly reference AA samples. For example, when using "branched path tracing" the there are many references to AA samples. The sample count input even has the tool tip "AA samples".
Since I'm not a developer, I'll leave the decision up to others and what to do with this.
As far as I know there's no setting in Blender called AA Samples anymore. If you find it let me know please.
Can't clean up anything without cleaning up everything. That's why it stays so messy. If we're ok with having it named different things all over the place then whatever. The vast majority of people aren't using branched path rendering on the CPU. Start changing the documentation and let the coders figure out what to do with their own deprecated names.
Added subscriber: @Blendify
This is technically correct. The documentation for this could be improved, however.
See the Arnold rendering docs for example: https://docs.arnoldrenderer.com/display/A5AFMUG/Samples
The Arnold documentation has absolutely nothing to do with this. The samples setting in Blender is called Samples or better yet Render Samples and not AA Samples. This is an easy fix. Documentation should use the language used in the application (Blenderin this case) for clarity and simplicity.
Branched path tracing still refers to AA samples. The tool tip for the "Render" sample count and "Viewport" sample count is "AA samples". The total sample count section at the end of the "Sub-sampling" section directly refers to AA samples. All the tool tips for the sub-samples also refer to AA samples as well as the documentation. The python handle for the branched path tracing sample count is also "cycles.aa_samples".
As for your comment about a vast majority of people not using branched path tracing, it's probably true, especially with OptiX not supporting it.
As for why other things in the documentation and Blender refer to AA samples like adaptive sampling, I can only assume it's to accommodate for branch path tracing and because this is how Cycles works internally.
I personally believe referring to samples as AA samples can still be useful. For example, if you were to render out a scene that consisted of only direct lighting and infinitely small point lights, you could render the scene in 1 sample and get all the lighting information with no noise. Yet increasing the sample count to 8 or 32, although not helping with mitigating noise, still provides a better images due to the anti-aliasing.
As for what the documentation should refer to. I will leave that decision up to the Blender developers. However, looking at the manual page you linked, I believe AA samples is referred to in all the right places. It's only ever talked about in reference to branched path tracing with the exception of the adaptive sampling feature.
The documentation doesn't even say what AA stands for.
It's implied with:
That part could be clearer. But to me it makes sense. Once again, I will leave the decision up to the Blender developers.
Changed status from 'Needs Triage' to: 'Archived'
I do not think this is an issue, technically regular samples are AA samples. For the adaptive sampling it important to note from the tooltip that it terminates the AA samples and not the subsamples so I think that is fine also.