Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1324)

Issue 5488046: Gamma Node for Cycles (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 5 months ago by dfelinto
Modified:
12 years, 5 months ago
Reviewers:
brechtvl, ZanQdo, bf-codereview
Base URL:
https://svn.blender.org/svnroot/bf-blender/trunk/blender/
Visibility:
Public.

Description

Gamma Node. I tried to use powr instead of if (.. > 0) powf, but didn't work. No harm I guess. For after bcon5. This node is not so useful for 'materials' but I believe it comes handy for textures input. Although it's simple to do it with the math node. *** committed on rev. 42671

Patch Set 1 #

Patch Set 2 : fixes from Lukas Toenne (no need for DNA and rna) + some cleanup #

Patch Set 3 : No real change in the code, simply update to help if someone wants to test. trunk ready I think #

Unified diffs Side-by-side diffs Delta from patch set Stats (+185 lines, -1 line) Patch
intern/cycles/app/cycles_xml.cpp View 1 2 1 chunk +3 lines, -0 lines 0 comments Download
intern/cycles/blender/blender_shader.cpp View 1 2 1 chunk +4 lines, -0 lines 0 comments Download
intern/cycles/kernel/CMakeLists.txt View 1 2 1 chunk +1 line, -0 lines 0 comments Download
intern/cycles/kernel/osl/nodes/CMakeLists.txt View 1 2 1 chunk +1 line, -0 lines 0 comments Download
intern/cycles/kernel/osl/nodes/node_gamma.osl View 1 2 1 chunk +33 lines, -0 lines 0 comments Download
intern/cycles/kernel/svm/svm.h View 1 2 2 chunks +4 lines, -0 lines 0 comments Download
intern/cycles/kernel/svm/svm_gamma.h View 1 2 1 chunk +37 lines, -0 lines 0 comments Download
intern/cycles/kernel/svm/svm_types.h View 1 2 1 chunk +2 lines, -1 line 0 comments Download
intern/cycles/render/nodes.h View 1 2 1 chunk +5 lines, -0 lines 0 comments Download
intern/cycles/render/nodes.cpp View 1 2 1 chunk +27 lines, -0 lines 0 comments Download
source/blender/blenkernel/BKE_node.h View 1 2 1 chunk +1 line, -0 lines 0 comments Download
source/blender/blenkernel/intern/node.c View 1 2 1 chunk +1 line, -0 lines 0 comments Download
source/blender/makesrna/intern/rna_nodetree_types.h View 1 2 1 chunk +1 line, -0 lines 0 comments Download
source/blender/nodes/CMakeLists.txt View 1 2 1 chunk +1 line, -0 lines 0 comments Download
source/blender/nodes/NOD_shader.h View 1 2 1 chunk +1 line, -0 lines 0 comments Download
source/blender/nodes/shader/nodes/node_shader_gamma.c View 1 2 1 chunk +63 lines, -0 lines 0 comments Download

Messages

Total messages: 10
dfelinto
12 years, 5 months ago (2011-12-12 16:56:44 UTC) #1
dfelinto
No real change in the code, simply update to help if someone wants to test. ...
12 years, 5 months ago (2011-12-16 18:47:02 UTC) #2
brechtvl
LGTM.
12 years, 5 months ago (2011-12-16 20:01:19 UTC) #3
zanqdo_gmail.com
I was testing the gamma node and noticed a discrepancy in the results from bringing ...
12 years, 5 months ago (2011-12-17 09:31:15 UTC) #4
dfelinto
The gamma node is doing a simple pow operation: color_out = color_in ** gamma (as ...
12 years, 5 months ago (2011-12-17 21:03:09 UTC) #5
zanqdo_gmail.com
Yes yes, however they are NOT consistent, there are slight differences, so I'm wondering what ...
12 years, 5 months ago (2011-12-18 08:02:02 UTC) #6
brechtvl
Gamma 2.2 and 1.0/2.2 should cancel each other out. However, Blender sRGB <=> linear conversion ...
12 years, 5 months ago (2011-12-18 12:17:50 UTC) #7
dfelinto
Brecht, is this to approximate the result from a real gamma curve? On 2011/12/18 12:17:50, ...
12 years, 5 months ago (2011-12-18 20:00:59 UTC) #8
brechtvl
The purpose of this scene linear color space in Blender is to roughly match physical ...
12 years, 5 months ago (2011-12-18 20:24:34 UTC) #9
zanqdo_gmail.com
12 years, 5 months ago (2011-12-19 12:20:27 UTC) #10
Ah! so they are indeed different algorithms. Humf I guess it's not a
problem atm

cheers

Daniel Salazar
3Developer.com


On Sun, Dec 18, 2011 at 2:24 PM, <brechtvanlommel@gmail.com> wrote:

> The purpose of this scene linear color space in Blender is to roughly
> match physical units of light, while sRGB is supposed to be a color
> space to display that on the screen. That's only an approximation in
> various ways, but it's more like a gamma curve is an even simpler
> approximation to this type of transform.
>
>
> On 2011/12/18 20:00:59, dfelinto wrote:
>
>> Brecht, is this to approximate the result from a real gamma curve?
>>
>
>
http://codereview.appspot.com/**5488046/<http://codereview.appspot.com/5488046/>
>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b