Page MenuHome
Create Task
Maniphest T48064

Smoke-cycles-OpenvVDB Adding velocity attribute crashes render
Closed, ResolvedPublic
Actions

Description

System Information
Operating system and graphics card

Blender Version
Broken: 673ddd4

Short description of error
Set that atribute and crash on render

Exact steps for others to reproduce the error
1-

smoke.blend97 KBDownload

2-try render (it should crash)
3-open again
4-change velocity to color or density
5-render (now shouldn't crash on render)

Revisions and Commits

rC Cycles
rCd79f2386e0d8 Add missing velocity attribute to builtin image loader
rB Blender
rB951fad2baa59 Cycles: Add missing velocity attribute to builtin image loader

Related Objects

Event Timeline

Nahuel Belich (belich) created this task.Apr 6 2016, 1:36 AM
Nahuel Belich (belich) raised the priority of this task from to 90.
Nahuel Belich (belich) updated the task description. (Show Details)
Nahuel Belich (belich) added a project: BF Blender.
Nahuel Belich (belich) edited a custom field.
Nahuel Belich (belich) added a subscriber: Nahuel Belich (belich).
jack (burton) changed the task status from Unknown Status to Resolved.Apr 6 2016, 1:55 AM
jack (burton) claimed this task.
Kévin Dietrich (kevindietrich) changed the task status from Resolved to Unknown Status.Apr 6 2016, 6:50 AM
Kévin Dietrich (kevindietrich) added a subscriber: Kévin Dietrich (kevindietrich).

@jack (burton), please avoid closing task at random with no explantation as to why you're closing this.

Kévin Dietrich (kevindietrich) removed jack (burton) as the assignee of this task.Apr 6 2016, 6:51 AM
Kévin Dietrich (kevindietrich) added a subscriber: jack (burton).
Kévin Dietrich (kevindietrich) lowered the priority of this task from 90 to 50.Apr 6 2016, 7:20 AM

The crash is due to an arithmetic exception thrown in the image interpolation code:

Program received signal SIGFPE, Arithmetic exception.

0x00000000032b46d1 in wrap_periodic (width=0, x=-1, this=<optimized out>)
    at /home/kevin/src/blender-git/blender/intern/cycles/kernel/./kernel_compat_cpu.h:113
113			x %= width;

Now the things is that only 'color', 'heat' and 'density' attributes/fields are supported in Cycles, but for some reason the 'velocity' attribute is still added and it crashes due to uninitialized data (would expect to have a nice pink/magenta volume instead, as for missing textures or images).

Kévin Dietrich (kevindietrich) added a project: Cycles.Apr 6 2016, 7:21 AM
Sergey Sharybin (sergey) changed the task status from Unknown Status to Resolved by committing rB951fad2baa59: Cycles: Add missing velocity attribute to builtin image loader.Apr 12 2016, 1:26 PM
Sergey Sharybin (sergey) added a commit: rB951fad2baa59: Cycles: Add missing velocity attribute to builtin image loader.
Sergey Sharybin (sergey) added a commit: rCd79f2386e0d8: Add missing velocity attribute to builtin image loader.Apr 13 2016, 9:54 AM