Undo re-orders dope sheet items which have cyclic dependencies #38491

New Issue

cmto · 2014-02-05T00:44:06+01:00

cmto commented

2014-02-05 00:44:06 +01:00

System Information
Win 7 x64, 560ti 1GB

Blender Version
Broken: 2.69.x
Worked: (Not sure it ever did)

If two items have a cyclic dependency with one another and you are undoing any keyframe changes on the dope sheet, the cyclic dependent items swap places with each other on the dope sheet list.

Exact steps for others to reproduce the error
cubes1.blend
Right click the keyframe on frame 0 of the Cone object, Shift+D then drag over a couple of frames and place the duplicated keyframe. Then undo the change and you will notice the two Cube objects swap places on the dope sheet. Note the cone is not constrained by/to anything, but the cubes are 'child-of' each other.

PS. I'm aware the cyclic dependency in my example is one that you would never actually use, but even the unproblematic 'acceptable' cyclic dependencies cause this issue.

**System Information** Win 7 x64, 560ti 1GB **Blender Version** Broken: 2.69.x Worked: (Not sure it ever did) If two items have a cyclic dependency with one another and you are undoing any keyframe changes on the dope sheet, the cyclic dependent items swap places with each other on the dope sheet list. **Exact steps for others to reproduce the error** [cubes1.blend](https://archive.blender.org/developer/F76040/cubes1.blend) Right click the keyframe on frame 0 of the Cone object, Shift+D then drag over a couple of frames and place the duplicated keyframe. Then undo the change and you will notice the two Cube objects swap places on the dope sheet. Note the cone is not constrained by/to anything, but the cubes are 'child-of' each other. ![cubes.png](https://archive.blender.org/developer/F76035/cubes.png) PS. I'm aware the cyclic dependency in my example is one that you would never actually use, but even the unproblematic 'acceptable' cyclic dependencies cause this issue.

cmto commented

2014-02-05 00:44:06 +01:00

Changed status to: 'Open'

cmto commented

2014-02-05 00:44:06 +01:00

Added subscriber: @CHARLES-MURRAY

Joshua Leung was assigned by Sergey Sharybin

2014-02-06 08:58:45 +01:00

Sergey Sharybin commented

2014-02-06 08:58:45 +01:00

Added subscribers: @JoshuaLeung, @Sergey

Sergey Sharybin commented

2014-02-06 08:58:45 +01:00

@JoshuaLeung, seems dopesheet displays channels in order of how the bmain is sorted, is it expected behavior? This way any change which affects on dopesheet could reshuffle channels.. Does it make sense to sort channels alphabetically?

Joshua Leung commented

2014-04-28 07:38:08 +02:00

In short: This is one of those "won't fix" in the short-medium term things.

From a usability perspective, yes it's not that great. However, this is an expected downside to the way it is currently intended to work. Without implementing some kind of cache (shudders) for listing out all the ID blocks in the scene in the order that they need to appear here, there's not much we can do about this in the short term. Completing the depsgraph refactor will likely give us an opportunity to improve stuff a bit here, since we won't need to be sorting bases in the scene by dependency order, so things will hopefully become more stable (and/or may be likely to be able to return to alphabetical ordering or at least user-defined order).

In short: This is one of those "won't fix" in the short-medium term things. From a usability perspective, yes it's not that great. However, this is an expected downside to the way it is currently intended to work. Without implementing some kind of cache (*shudders*) for listing out all the ID blocks in the scene in the order that they need to appear here, there's not much we can do about this in the short term. Completing the depsgraph refactor will likely give us an opportunity to improve stuff a bit here, since we won't need to be sorting bases in the scene by dependency order, so things will hopefully become more stable (and/or may be likely to be able to return to alphabetical ordering or at least user-defined order).

Bastien Montagne commented

2014-09-06 12:03:49 +02:00

Changed status from 'Open' to: 'Archived'

Bastien Montagne closed this issue

2014-09-06 12:03:49 +02:00

Bastien Montagne commented

2014-09-06 12:03:49 +02:00

Added subscriber: @mont29

Bastien Montagne commented

2014-09-06 12:03:49 +02:00

Thanks for the report, but no point in keeping it open forever. Added a note in our TODO list.

Thanks for the report, but no point in keeping it open forever. Added a note in our [TODO list](http://wiki.blender.org/index.php/Dev:2.5/Source/Development/Todo/Animation#Animation_System).

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Undo re-orders dope sheet items which have cyclic dependencies #38491