Maniphest T101462

Regression : Geometry nodes: Time-dependent delete points is not taken into account by velocity pass on render
Closed, Resolved

Assigned To
Hans Goudey (HooglyBoogly)
Authored By
CHET (cheteron)
Sep 29 2022, 3:04 PM
Tags
  • Geometry Nodes
  • Nodes & Physics
  • Render & Cycles
  • BF Blender (3.3)
Subscribers
CHET (cheteron)
Christophe Leyder (shotalot)
Hans Goudey (HooglyBoogly)
Iliya Katueshenock (Moder)
Pratik Borhade (PratikPB2123)

Description

System Information
Operating system: Windows-10-10.0.19044-SP0 64 Bits
Graphics card: NVIDIA GeForce GTX 950M/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 512.15

Blender Version
Broken: version: 3.3.0, rB7a4a6ccad74f
Worked version: 3.3.0, rBd0a552b5c602
Worked: 3.4

Short description
Crash when rendering the particular GN tree if vector pass is enabled

Error:

Vendor: NVIDIA Corporation
Renderer: NVIDIA GeForce GTX 1060 6GB/PCIe/SSE2
Version: 4.5.0 NVIDIA 496.49
Context Version: 4.5
Array iterator out of range: (index 1128)

Play steps

  1. Create an object with geometry nodes.
  2. Create some way points geometry.
  3. Create Delete Geometry Node.
  4. Create the Scene Time and Index input nodes.
  5. Compare time (frame) and index and entry this as selection for delete geometry node.
  6. Put points to delete geometry node
  7. Turn on the Vector render pass output.
  8. Start rendering a random frame until a crash occurs.

Related Objects

Mentioned In
T101496: RE_engine_gpu_context_destroy(), at 'BLI_thread_is_main()'
Mentioned Here
rB21ed3b325888: Fix T101025: Cycles motion blur crash with changing point cloud size

Event Timeline

CHET (cheteron) created this task.Sep 29 2022, 3:04 PM
CHET (cheteron) added a comment.Sep 29 2022, 3:19 PM

Crash when used Vector pass

Iliya Katueshenock (Moder) added a subscriber: Iliya Katueshenock (Moder).EditedSep 29 2022, 5:34 PM

Checking...

Iliya Katueshenock (Moder) renamed this task from Blender crash to Geometry nodes: Time-dependent offset is not taken into account by velocity pass.Sep 29 2022, 5:53 PM
Iliya Katueshenock (Moder) renamed this task from Geometry nodes: Time-dependent offset is not taken into account by velocity pass to Geometry nodes: Time-dependent delete is not taken into account by velocity pass.
Iliya Katueshenock (Moder) changed the task status from Needs Triage to Confirmed.
Iliya Katueshenock (Moder) updated the task description.
Iliya Katueshenock (Moder) edited projects, added Geometry Nodes, Nodes & Physics, Render & Cycles; removed BF Blender.
Iliya Katueshenock (Moder) renamed this task from Geometry nodes: Time-dependent delete is not taken into account by velocity pass to Regression: Geometry nodes: Time-dependent delete is not taken into account by velocity pass.Sep 29 2022, 6:01 PM
Iliya Katueshenock (Moder) updated the task description.
Iliya Katueshenock (Moder) updated the task description.Sep 29 2022, 6:05 PM
Iliya Katueshenock (Moder) updated the task description.Sep 29 2022, 6:07 PM
Iliya Katueshenock (Moder) updated the task description.
Iliya Katueshenock (Moder) updated the task description.Sep 29 2022, 6:12 PM
Iliya Katueshenock (Moder) triaged this task as High priority.Sep 30 2022, 2:16 PM
Iliya Katueshenock (Moder) updated the task description.
Hans Goudey (HooglyBoogly) changed the task status from Confirmed to Needs Information from User.Oct 5 2022, 6:45 PM
Hans Goudey (HooglyBoogly) added a subscriber: Hans Goudey (HooglyBoogly).

This report is missing steps to reproduce the problem.

CHET (cheteron) added a comment.Oct 5 2022, 6:55 PM
In T101462#1428083, @Hans Goudey (HooglyBoogly) wrote:

This report is missing steps to reproduce the problem.

Did you read the whole topic, or do you always close the task, even if the blender crashes? Is this a new developer policy?
I'm an artist, not a beta-tester, and I can't figure out what exactly is wrong.

Iliya Katueshenock (Moder) updated the task description.Oct 6 2022, 8:40 AM

Sorry, totally forgot about this.
I thought that a description of the problem and a simple file would be enough (later I found out that it wasn’t, but I forgot about this report), but here is the description

Iliya Katueshenock (Moder) updated the task description.Oct 6 2022, 8:42 AM
Christophe Leyder (shotalot) added a subscriber: Christophe Leyder (shotalot).Oct 6 2022, 1:35 PM

This is a known limitation for deformation blur in any renderer, Cycles doesn't know how to calculate the velocity because you are changing the point count frame-by-frame. Unless theres a way to calculate the velocity directly in geometry nodes using some sort of Point ID attribute, but I dont think there is a way to do this currently.

Iliya Katueshenock (Moder) added a comment.EditedOct 6 2022, 2:02 PM

@CHET (cheteron) Can you still reproduce this in the latest blender build?

This is a known limitation for deformation blur in any renderer, Cycles doesn't know how to calculate the velocity because you are changing the point count frame-by-frame. Unless theres a way to calculate the velocity directly in geometry nodes using some sort of Point ID attribute, but I dont think there is a way to do this currently.

It's ok if it's limited, but it shouldn't crash

CHET (cheteron) added a comment.Oct 6 2022, 4:06 PM
In T101462#1428522, @Iliya Katueshenock (Moder) wrote:

@CHET (cheteron) Can you still reproduce this in the latest blender build?

Crash

Iliya Katueshenock (Moder) renamed this task from Regression: Geometry nodes: Time-dependent delete is not taken into account by velocity pass to Regression : Geometry nodes: Time-dependent delete points is not taken into account by velocity pass on render.Oct 6 2022, 5:35 PM
Iliya Katueshenock (Moder) updated the task description.

Oh, sorry, I completely forgot about the difference between 3.3 and 3.4. Looks like it's only a 3.3 issue

Iliya Katueshenock (Moder) updated the task description.Oct 6 2022, 5:36 PM
Iliya Katueshenock (Moder) updated the task description.
Iliya Katueshenock (Moder) added a project: BF Blender (3.3).Oct 6 2022, 5:39 PM
Iliya Katueshenock (Moder) updated the task description.
Iliya Katueshenock (Moder) updated the task description.Oct 6 2022, 5:44 PM
Pratik Borhade (PratikPB2123) changed the task status from Needs Information from User to Confirmed.EditedOct 7 2022, 6:52 AM
Pratik Borhade (PratikPB2123) updated the task description.
Pratik Borhade (PratikPB2123) added a subscriber: Pratik Borhade (PratikPB2123).

Seems crash is replicable in 3.3 only. I think we would need to find the resolving commit from master branch.

Hans Goudey (HooglyBoogly) closed this task as Resolved.Oct 7 2022, 7:35 PM
Hans Goudey (HooglyBoogly) claimed this task.

I added rB21ed3b325888 to the 3.3 maintenance task, so this should be fixed in 3.3.2. I'll close this as resolved since there isn't more to do for now though. Thanks for the report.