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

Issue 12740045: Fix for #36468, "Buffer Groups" option changes compositing output

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 8 months ago by lukas.toenne1
Modified:
10 years, 8 months ago
Reviewers:
bf-codereview, jbkkavt, j.bakker
Base URL:
https://svn.blender.org/svnroot/bf-blender/trunk/blender/
Visibility:
Public.

Description

https://projects.blender.org//tracker/?func=detail&atid=498&aid=36468&group_id=9 Problem is that the read/write buffer operations only work with actual image inputs. If a singular value is used as group input no actual buffer will be created, the write operation does not schedule any chunks and the ReadBufferOperation subsequently returns zero (MemoryBuffer::read). The fix uses the (0,0) resolution to detect single value input of the WriteBufferOperation. The actual resolution is then clamped to (1,1) to ensure we have a single pixel to store the value in. A m_single_value flag is also set, so we can reliably distinguish this from genuine image resolutions without having to check m_width/m_height later on. The ReadBufferOperation copies this flag from the associated WriteBufferOperation and if set will always return the single value from pixel (0,0).

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+34 lines, -2 lines) Patch
source/blender/compositor/operations/COM_ReadBufferOperation.h View 1 chunk +1 line, -0 lines 0 comments Download
source/blender/compositor/operations/COM_ReadBufferOperation.cpp View 3 chunks +15 lines, -2 lines 0 comments Download
source/blender/compositor/operations/COM_WriteBufferOperation.h View 2 chunks +3 lines, -0 lines 0 comments Download
source/blender/compositor/operations/COM_WriteBufferOperation.cpp View 1 chunk +15 lines, -0 lines 0 comments Download

Messages

Total messages: 3
lukas.toenne1
10 years, 8 months ago (2013-08-15 06:25:43 UTC) #1
jbkkavt
On 2013/08/15 06:25:43, lukas.toenne1 wrote: I got the issue, but not sure if we have ...
10 years, 8 months ago (2013-08-17 14:55:54 UTC) #2
jbkkavt
10 years, 8 months ago (2013-08-17 14:56:26 UTC) #3
On 2013/08/17 14:55:54, jbkkavt wrote:
> On 2013/08/15 06:25:43, lukas.toenne1 wrote:
> 
> I got the issue, but not sure if we have introduced other issues with this
fix.
> opencl buffers are currently not checked on their dimensions.
> The specs say it should work, as long as the right samplers are used.
> 
> Jeroen

We will need to find a way to test this.
Sign in to reply to this message.

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