Blender Internal: Render tiles do not appear until render is finished if Save Buffers enabled #39029

New Issue

Zauber Paracelsus · 2014-03-07T22:31:54+01:00

Zauber Paracelsus commented

2014-03-07 22:31:54 +01:00

System Information
64bit Linux Mint 16
Linux Kernel 3.11.0
nVidia GTX 650 Ti, driver 319.32

Blender Version
Broken: 2.70 Release Candidate 1
Worked: 2.69.0 r60991

Short description of error
When you Render a scene, the rendering tiles appear completely blank and remain so until the entire image is rendered. This only occurs if the "Save Buffers"

Exact steps for others to reproduce the error

Create a scene
Enable the "Save Buffers" setting under Performance.
Render

**System Information** 64bit Linux Mint 16 Linux Kernel 3.11.0 nVidia GTX 650 Ti, driver 319.32 **Blender Version** Broken: 2.70 Release Candidate 1 Worked: 2.69.0 r60991 **Short description of error** When you Render a scene, the rendering tiles appear completely blank and remain so until the entire image is rendered. This only occurs if the "Save Buffers" **Exact steps for others to reproduce the error** 1) Create a scene 2) Enable the "Save Buffers" setting under Performance. 3) Render

Zauber Paracelsus commented

2014-03-07 22:31:54 +01:00

Changed status to: 'Open'

Zauber Paracelsus commented

2014-03-07 22:31:54 +01:00

Added subscriber: @ZauberParacelsus

Joel Godin commented

2014-03-08 01:19:39 +01:00

Added subscriber: @joel.godin

Joel Godin commented

2014-03-08 01:19:39 +01:00

Not seeing that on OSX 10.9.2

Zauber Paracelsus commented

2014-03-08 02:58:01 +01:00

Apparently I submitted before I was finished writing. The last sentence should read "This only occurs if the Save Buffers setting is enabled, under the Performance section of the Render tab."

Thomas Dinges commented

2014-03-08 13:10:35 +01:00

Added subscriber: @ThomasDinges

Thomas Dinges commented

2014-03-08 13:10:35 +01:00

Does not happen with Cycles, only Blender Internal.

Confirmed on Windows 7 x64, with latest master.

Does not happen with Cycles, only Blender Internal. Confirmed on Windows 7 x64, with latest master.

Brecht Van Lommel was assigned by Bastien Montagne

2014-03-08 18:00:35 +01:00

Bastien Montagne commented

2014-03-08 18:00:35 +01:00

Added subscriber: @mont29

Bastien Montagne commented

2014-03-08 18:00:35 +01:00

@ThomasDinges Even though it is present in Cycles UI, afaict Cycles completely ignores it, so I guess it could be removed from here?

Regarding BI, imho it’s not a bug - current code writes tiles into temp EXR files, and re-read it once render is finished, to display it in Blender. Reading EXR files after every tile is down could be done I guess, but not easily, and would kinda break the main purpose of this option (as we would have to keep one layer in memory during whole render). Assigning to Brecht anyway, to get its advice here!

@ThomasDinges Even though it is present in Cycles UI, afaict Cycles completely ignores it, so I guess it could be removed from here? Regarding BI, imho it’s not a bug - current code writes tiles into temp EXR files, and re-read it once render is finished, to display it in Blender. Reading EXR files after every tile is down could be done I guess, but not easily, and would kinda break the main purpose of this option (as we would have to keep one layer in memory during whole render). Assigning to Brecht anyway, to get its advice here!

Brecht Van Lommel commented

2014-03-08 18:42:08 +01:00

Save buffers works for Cycles. If I had to guess then I think this is related to color management. It should be possible to have this preview work for BI too.

Brecht Van Lommel removed their assignment 2014-03-08 18:42:23 +01:00

Zauber Paracelsus commented

2014-03-08 18:58:36 +01:00

@mont29: It is an undesirable behavior that did not occur prior to 2.70. So it is not only a bug, it is also a regression.

Bastien Montagne self-assigned this 2014-03-08 19:00:30 +01:00

Bastien Montagne commented

2014-03-08 19:00:30 +01:00

Uh… well, then, the least I can do is fix it :/

Zauber Paracelsus commented

2014-03-08 19:03:18 +01:00

@mont29: Sorry if I came down a little hard. Due to previous bad experiences in life, I get agitated whenever I feel like something I've said was ignored. So, my apologies.

Bastien Montagne commented

2014-03-11 10:55:44 +01:00

Added subscriber: @Sergey

Bastien Montagne commented

2014-03-11 10:55:44 +01:00

Gah… spent hours on this one, for a rather poor result… :/ So far:

Issue appeared in c5d7ea091 (but I do not see what’s the culprit it in this commit).
Issue is caused by temp renderresults used to generate preview (in e.g. RE_AcquireResultImage()) having neither rectf nor rect32 buffer, when using SaveBuffer.
When saving renderlayers in EXR instead of mem, code seems to allocate a dedicated buffer (in render_result_new(), rl->display_buffer), but never use it anywhere, as far as I can tell?

@Sergey, I think (hope!) you know/understand this stuff better than me, maybe it wouldn't be that hard for you to find a solution (or at least, understand what went wrong)?

Gah… spent hours on this one, for a rather poor result… :/ So far: * Issue appeared in c5d7ea091 (but I do not see what’s the culprit it in this commit). * Issue is caused by temp renderresults used to generate preview (in e.g. `RE_AcquireResultImage()`) having neither `rectf` nor `rect32` buffer, when using SaveBuffer. * When saving renderlayers in EXR instead of mem, code seems to allocate a dedicated buffer (in `render_result_new()`, `rl->display_buffer`), but never use it anywhere, as far as I can tell? @Sergey, I think (hope!) you know/understand this stuff better than me, maybe it wouldn't be that hard for you to find a solution (or at least, understand what went wrong)?

blender-admin commented

2014-03-11 14:56:03 +01:00

This issue was referenced by blender/blender-addons-contrib@89c793f70f

This issue was referenced by blender/blender-addons-contrib@89c793f70f2cc65c7ae2a903a8b747a3fb799f38

blender-admin commented

2014-03-11 14:56:03 +01:00

This issue was referenced by 89c793f70f

This issue was referenced by 89c793f70f2cc65c7ae2a903a8b747a3fb799f38

Sergey Sharybin commented

2014-03-11 14:56:32 +01:00

Changed status from 'Open' to: 'Resolved'

Sergey Sharybin closed this issue