Assert when moving object parented to curve vertex #57583

Closed
opened 2018-11-02 16:06:30 +01:00 by Dalai Felinto · 8 comments

Blender Version
Broken: f2873b3ce5 (blender2.8 HEAD)
Worked: 2.79

Short description of error
Assert when moving object parented to a curve vertex.

Exact steps for others to reproduce the error

  • Add an empty and a curve
  • With the empty selected, select the curve, go to edit mode and create vertex parent (Ctrl+P)
  • Go to object mode
    Or open simple file:
    curve_vertex_parent.blend

Now Move the empty:

BLI_assert failed: //source/blender/blenkernel/intern/object.c:2129, give_parvert(), at 'par->runtime.curve_cache != ((void *)0)'

Full backtrace: P816

**Blender Version** Broken: f2873b3ce540 (blender2.8 HEAD) Worked: 2.79 **Short description of error** Assert when moving object parented to a curve vertex. **Exact steps for others to reproduce the error** * Add an empty and a curve * With the empty selected, select the curve, go to edit mode and create vertex parent (Ctrl+P) * Go to object mode *Or open simple file:* [curve_vertex_parent.blend](https://archive.blender.org/developer/F5337776/curve_vertex_parent.blend) Now Move the empty: `BLI_assert failed: //source/blender/blenkernel/intern/object.c:2129, give_parvert(), at 'par->runtime.curve_cache != ((void *)0)'` Full backtrace: [P816](https://archive.blender.org/developer/P816.txt)
Author
Owner

Added subscriber: @dfelinto

Added subscriber: @dfelinto
Sergey Sharybin was assigned by Dalai Felinto 2018-11-02 16:07:24 +01:00
Author
Owner

Added subscriber: @Sergey

Added subscriber: @Sergey
Author
Owner

@Sergey want to tackle this one? It is probably some deg relation missing.

@Sergey want to tackle this one? It is probably some deg relation missing.

Added subscriber: @brecht

Added subscriber: @brecht

This is nothing to do with the relations, it is to do with the copies and access to evaluated/original data (internal project codename "COW FIASCO" :)

Here is a patch which fixes the issue, but we might want to reconsider something on a global scale, to make sure things like studio/blender-studio#55494 also work.

Wouldn't mind discussing it with @brecht here.

#57583.patch

This is nothing to do with the relations, it is to do with the copies and access to evaluated/original data (internal project codename "COW FIASCO" :) Here is a patch which fixes the issue, but we might want to reconsider something on a global scale, to make sure things like studio/blender-studio#55494 also work. Wouldn't mind discussing it with @brecht here. [#57583.patch](https://archive.blender.org/developer/F5582842/T57583.patch)

The proposed patch is not correct, the parent solving is used from dependency graph evaluation as well (and not only for the transformation tools and such). Testing alternative solution now.

The proposed patch is not correct, the parent solving is used from dependency graph evaluation as well (and not only for the transformation tools and such). Testing alternative solution now.

This issue was referenced by 13f28798de

This issue was referenced by 13f28798de5d501ed8867bbc0d7a25ca6f1dbe9b

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Sign in to join this conversation.
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: blender/blender#57583
No description provided.