Page MenuHome
Differential D832

Cycles: Experiment with low light threshold
AbandonedPublic
Actions

Authored by Sergey Sharybin (sergey) on Oct 17 2014, 4:11 PM.

Details

Reviewers
Brecht Van Lommel (brecht)
Thomas Dinges (dingto)
Martijn Berger (juicyfruit)
Summary

The idea is to skip evaluation of lamps when their contribution is too low.

This easily saves up to 15% of render time in the secret agent shot without
affecting the light of the scene much.

Diff Detail

Repository
rB Blender
Branch
cycles_low_light_threshold

Event Timeline

Sergey Sharybin (sergey) updated this revision to Diff 2731.Oct 17 2014, 4:11 PM
Sergey Sharybin (sergey) retitled this revision from to Cycles: Experiment with low light threshold.
Sergey Sharybin (sergey) added reviewers: Brecht Van Lommel (brecht), Martijn Berger (juicyfruit), Thomas Dinges (dingto).
Sergey Sharybin (sergey) updated this object.
Sergey Sharybin (sergey) added subscribers: Ton Roosendaal (ton), Pablo Vazquez (pablovazquez).
Sergey Sharybin (sergey) added a comment.Oct 17 2014, 4:12 PM

I'm not sure using pdf as a contribution is correct here. On the one hand it's quite correct on the other it's a bit arguable..

Ronny G (nutel) added a subscriber: Ronny G (nutel).Oct 17 2014, 4:57 PM
Derek Barker (lordodin) added a subscriber: Derek Barker (lordodin).Oct 18 2014, 7:17 AM
Vasyl Pidgirskiy (sdilmob) added a subscriber: Vasyl Pidgirskiy (sdilmob).Jan 2 2015, 10:06 PM
Sergey Sharybin (sergey) planned changes to this revision.Jan 15 2015, 9:20 AM
Kitt Zwovic (gandalf3) added a subscriber: Kitt Zwovic (gandalf3).Feb 4 2015, 8:59 AM
Carlo Andreacchio (candreacchio) added a subscriber: Carlo Andreacchio (candreacchio).Feb 4 2015, 11:46 PM
Jason Clarke (jtheninja) added a subscriber: Jason Clarke (jtheninja).Mar 8 2015, 10:16 PM
Sergey Sharybin (sergey) mentioned this in T46258: Cycles plans for Blender 2.8x project.Sep 25 2015, 8:55 AM
Brecht Van Lommel (brecht) mentioned this in D2217: Cycles: Add optional probabilistic termination of light samples based on their expected contribution.Sep 10 2016, 11:13 PM
Sergey Sharybin (sergey) abandoned this revision.Dec 2 2016, 5:00 PM

This is now implemented in a proper way.

Revision Contents
Changeset List

PathSize
intern/
cycles/
blender/
addon/
7 lines
1 line
2 lines
kernel/
8 lines
8 lines
5 lines
render/
2 lines
5 lines

Diff 2731

View Options

intern/cycles/blender/addon/properties.py

Loading...
View Options

intern/cycles/blender/addon/ui.py

Loading...
View Options

intern/cycles/blender/blender_sync.cpp

Loading...
View Options

intern/cycles/kernel/kernel_path_surface.h

Loading...
View Options

intern/cycles/kernel/kernel_path_volume.h

Loading...
View Options

intern/cycles/kernel/kernel_types.h

Loading...
View Options

intern/cycles/render/integrator.h

Loading...
View Options

intern/cycles/render/integrator.cpp

Loading...