Page MenuHome
Create Task
Maniphest T72783

"Persistent images" option crashes smoke animation render with Cycles
Closed, ResolvedPublicBUG
Actions

Description

System Information
Operating system: Windows-10-10.0.18362 64 Bits
Graphics card: GeForce GTX 1080/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 441.20

Blender Version
Broken: version: 2.81 (sub 16), branch: master, commit date: 2019-12-04 11:32, hash: rBf1aa4d18d49d

Short description of error
Using the "Persistent images" option in Render Properties/Performance/Final Render while rendering a smoke animation with Cycles crashes Blender

Exact steps for others to reproduce the error

  1. Start new "General" blender project
  2. With cube selected, click Objects/Quick Effects/Quick Smoke
  3. Change renderer to Cycles
  4. Enable "Persistent images" in Render Properties/Performance/Final Render
  5. Save the project somewhere so the smoke can be baked
  6. Bake the smoke in Physics Properties/Cache
  7. Change render resolution to 500x500 (to save time)
  8. Change render samples to 32 (to save time)
  9. Render animation

Crash within a few frames (I crashed on the 8th)

Revisions and Commits

rB Blender
rBadcc9d014cc2 Fluid: Moved grid reset loop for inner obstacle cells from blenkernel code into…
rB2ff3877f71fb Fluid: Moved grid reset loop for inner obstacle cells from blenkernel code into…

Related Objects

Event Timeline

Dominic Lacroix (kendoka15) created this task.Dec 30 2019, 6:16 AM
Richard Antalik (ISS) changed the task status from Needs Triage to Confirmed.Jan 1 2020, 11:52 PM
Richard Antalik (ISS) added a project: Nodes & Physics.
Richard Antalik (ISS) added a subscriber: Richard Antalik (ISS).

In debug build this fails on bake step

/* Obstacle cells should not contain any velocity from the smoke simulation. */
for (z = 0; z < mds->res[0] * mds->res[1] * mds->res[2]; z++) {
  if (obstacles[z] & 2) /* Mantaflow convention: FlagObstacle. */

with obstacles being NULL

Can also confirm crash in release build during render.

Bastien Montagne (mont29) changed the subtype of this task from "Report" to "Bug".Jan 15 2020, 10:57 AM
Bastien Montagne (mont29) added projects: Cycles, Render & Cycles.
Bastien Montagne (mont29) added subscribers: Sebastián Barschkis (sebbas), Bastien Montagne (mont29).

@Richard Antalik (ISS) new issue with current master and mantaflow most likely should be a completely different report, they are not related at all I’d bet.

Sebastián Barschkis (sebbas) added a comment.Jan 15 2020, 11:03 AM

I am currently looking into this report T72894 which seems to be the same issue.

Philipp Oeser (lichtwerk) added a subscriber: Philipp Oeser (lichtwerk).Jan 15 2020, 11:31 AM

Not sure which data cycles actually keeps, but I assume there are just cases where this just makes more sense to being forced OFF?
(not sure these are really related, but it is forced OFF for OSL, see rBe0ca72735d58: Further fix for #34121: OSL + persistent images could crash in some cases still. and there is also T71004 which I assume would benefit from forcing Persisten Images OFF...)

Sebastián Barschkis (sebbas) closed this task as Resolved by committing rB2ff3877f71fb: Fluid: Moved grid reset loop for inner obstacle cells from blenkernel code into….Jan 15 2020, 3:52 PM
Sebastián Barschkis (sebbas) claimed this task.
Sebastián Barschkis (sebbas) added a commit: rB2ff3877f71fb: Fluid: Moved grid reset loop for inner obstacle cells from blenkernel code into….
Sebastián Barschkis (sebbas) added a commit: rBadcc9d014cc2: Fluid: Moved grid reset loop for inner obstacle cells from blenkernel code into….Jan 15 2020, 4:30 PM