Increase Eevee volume samples to allow rendering animations
Needs RevisionPublic
Actions

Authored by Gottfried Hofmann (gottfried) on Jul 28 2019, 10:45 AM.

Details

Reviewers

Brecht Van Lommel (brecht)
Clément Foucault (fclem)

Summary

Eevee currently allows on 256 volume samples which can lead to severe banding issues even with optimized start and end for the volume frustum.

Increasing volumetric_samples fixes those issues.

Demo and file in this twitter thread: https://twitter.com/StefanPWinc/status/1154973548508778496

Another example of banding issues: https://twitter.com/brentpatterson/status/1154042824045678593

The shadow samples are also increased in this patch because I noticed that increasing them changes the look of the clouds, thus the currently limit is too small as well...

Diff Detail

Repository: rB Blender

Event Timeline

Gottfried Hofmann (gottfried) created this revision.Jul 28 2019, 10:45 AM

Gottfried Hofmann (gottfried) added projects: Eevee, User Interface.

Fin O'Riordan (fin.eskimo) awarded a token.Jul 28 2019, 12:22 PM

Fin O'Riordan (fin.eskimo) added a subscriber: Fin O'Riordan (fin.eskimo).

Marco (nacioss) awarded a token.Jul 28 2019, 3:15 PM

Julian Eisel (Severin) edited reviewers, added: Clément Foucault (fclem); removed: Dalai Felinto (dfelinto), Bastien Montagne (mont29).Jul 28 2019, 4:19 PM

This is probably fine if it doesn't e.g. cause driver timeouts if this value goes too high. Will leave this to @Clément Foucault (fclem) to judge, but he has time off this week so might be a bit until he looks at this.

Thomas Murphy (tmurphy) awarded a token.Jul 30 2019, 4:28 PM

Thomas Murphy (tmurphy) added a subscriber: Thomas Murphy (tmurphy).

noki paike (amonpaike) awarded a token.Jul 31 2019, 10:23 AM

Andres Stephens (Draise) awarded a token.Nov 14 2019, 7:07 PM

The reason 256 is the limit is because of it is the mimum maximum 3D texture size ensured by the OpenGL 3.3 spec.
Opening a file on any platform should not result in unsupported behavior.

If we are to uncap this setting I would like to have some safe-check and error message if the resolution is not supported.

This revision now requires changes to proceed.Mar 23 2020, 9:59 PM

Büke Beyond (beyond) awarded a token.Jan 2 2021, 7:20 PM

Revision Contents
Changeset List

Path

Size

source/

blender/

makesrna/

intern/

rna_scene.c

4 lines

Diff 16637

View Options

source/blender/makesrna/intern/rna_scene.c

Increase Eevee volume samples to allow rendering animationsNeeds RevisionPublicActions