0001141: Tonemapping/clamping defaults

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0001141

LuxRender

Core

public

2011-08-25 11:45

2012-08-21 13:17

Reporter

J the Ninja

Assigned To

moure

Priority

normal

Severity

minor

Reproducibility

N/A

Status

assigned

Resolution

open

Platform

OS Version

Product Version

0.8

Target Version

Fixed in Version

Summary

0001141: Tonemapping/clamping defaults

Description

From my initial post in this thread: http://www.luxrender.net/forum/viewtopic.php?f=8&t=6435 [^]

"At the moment, LuxRender defaults to the Reinhard tonemapper and sets the LDR clamp mode to "lum". Ostensibly, this is to mimic an auto-exposure camera, except the auto-linear+"cut" combo actually behaves much more like an auto-exposure camera than Reinhard does. Reinhard is a bit more complex than that and often confuses new users with its behavior. We even have an entry in the FAQ for users who are surprised by how it reacts to changes in relative light brightness. And the behavior of the "lum" clamp mode is just plain weird, IMO.

I've been using auto-linear+cut as my defaults for awhile now (by setting them in my startup.blend file for lb25) and am beginning to wonder if it would be best to change the actual defaults used in the core and what the exporters set. So, I'd like to get people thoughts on two things:

1. Changing the default tonemapping kernel in the core and exporters to auto-linear instead of Reinhard.
2. Changing the default LDR clamp mode, again for both the core and exporters, to cut (clip individually) instead of lum (preserve luminosity)"

The idea didn't get a whole lot of comments, but most people I talked to (either in that thread or on IRC) expressed support for the idea. I propose implementing this for 0.9

Tags

No tags attached.

Mercurial Changeset #

Requires Documentation Update

Requires Exporter Update

LuxBlend, LuxBlend25, LuXSI, LuxMax, SU2LUX

Attached Files

Relationships

Notes
(0003160) jeanphi (administrator) 2011-08-31 05:47	I agree with the change. I can handle the core part even if that's the less useful one.

(0003189) J the Ninja (developer) 2011-09-06 18:12	I can handle the LuxBlend25 part myself. I think most exporter devs will go ahead and adapt if the core is changed, especially if the commit notes or a forum post mentions that they should. So far, no one has voiced opposition to the switch, perhaps we should just go ahead and do it?

(0003230) jeanphi (administrator) 2011-10-03 05:27	You can now proceed to update the exporters.

(0003231) J the Ninja (developer) 2011-10-03 10:01	LuxBlend25 has been updated

(0003245) J the Ninja (developer) 2011-11-19 23:32	Passing along the assignment baton, once each exporter is updated, pass it along. :)

Issue History
Date Modified	Username	Field	Change
2011-08-25 11:45	J the Ninja	New Issue
2011-08-31 05:47	jeanphi	Note Added: 0003160
2011-09-06 18:12	J the Ninja	Note Added: 0003189
2011-09-28 22:11	jeanphi	Assigned To	=> jeanphi
2011-09-28 22:11	jeanphi	Status	new => assigned
2011-10-03 05:27	jeanphi	Assigned To	jeanphi => J the Ninja
2011-10-03 05:27	jeanphi	Note Added: 0003230
2011-10-03 10:01	J the Ninja	Note Added: 0003231
2011-11-19 23:31	J the Ninja	Assigned To	J the Ninja => moure
2011-11-19 23:32	J the Ninja	Note Added: 0003245
2012-08-21 13:17	jeanphi	Target Version	1.0 =>