Make Instances Real: Keep Hierarchy option has no effect when duplicates are collection instances #66282

New Issue

Daniel · 2019-06-30T19:55:23+02:00

Daniel commented

2019-06-30 19:55:23 +02:00

System Information
Operating system: Windows-10-10.0.17134 64 Bits
Graphics card: GeForce GTX 1060 6GB/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 430.39

Blender Version
Broken: version: 2.80 (sub 74), branch: blender2.8, commit date: 2019-06-30 15:32, hash: 7c8bf77e13

Short description of error
When using the make instance real function the hierarchy of the objects gets lost even if the keep hierarchy option is ticked which affects both the grouping of the objects inside the original collection as well as any object parenting that is involved in the collection.

Exact steps for others to reproduce the error
In the uploaded blender file execute the "Make Instance Real" operation and ensure "Keep Hierarchy" is ticked.
keep hiearchy bug.blend

**System Information** Operating system: Windows-10-10.0.17134 64 Bits Graphics card: GeForce GTX 1060 6GB/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 430.39 **Blender Version** Broken: version: 2.80 (sub 74), branch: blender2.8, commit date: 2019-06-30 15:32, hash: `7c8bf77e13` **Short description of error** When using the make instance real function the hierarchy of the objects gets lost even if the keep hierarchy option is ticked which affects both the grouping of the objects inside the original collection as well as any object parenting that is involved in the collection. **Exact steps for others to reproduce the error** In the uploaded blender file execute the "Make Instance Real" operation and ensure "Keep Hierarchy" is ticked. [keep hiearchy bug.blend](https://archive.blender.org/developer/F7557475/keep_hiearchy_bug.blend)

Daniel commented

2019-06-30 19:55:23 +02:00

Added subscriber: @bestdani

Daniel changed title from ~~Make Instances Real: Keep Hiearchy option has no effect when duplicates are collection instances~~ to Make Instances Real: Keep Hierarchy option has no effect when duplicates are collection instances

2019-07-01 08:11:57 +02:00

Philipp Oeser commented

2019-07-01 10:33:36 +02:00

Added subscribers: @mont29, @lichtwerk

Bastien Montagne was assigned by Philipp Oeser

2019-07-01 10:33:36 +02:00

Philipp Oeser commented

2019-07-01 10:33:36 +02:00

asking @mont29 for advice here: not 100% sure how instances are handled in this case, but the way it is handled atm. looks a little weird [duplicating collection instance contents as well as collection instances as well?]

Will confirm for the time being...

asking @mont29 for advice here: not 100% sure how instances are handled in this case, but the way it is handled atm. looks a little weird [duplicating collection instance contents as well as collection instances as well?] Will confirm for the time being...

Bastien Montagne commented

2019-07-03 14:06:51 +02:00

The behavior of creating object copies is working as expected (Collection copies are mere empties, not duplicollections anymore).

The problem is with re-parenting, that one is broken for anything but simplest cases, and reading the code, has been broken for years (it's trying to get parent object's new copy using child persistent_id...). Still trying to figure out the proper solution here.

The behavior of creating object copies is working as expected (Collection copies are mere empties, not duplicollections anymore). The problem is with re-parenting, that one is broken for anything but simplest cases, and reading the code, has been broken for years (it's trying to get parent object's new copy using child `persistent_id`...). Still trying to figure out the proper solution here.

Bastien Montagne commented

2019-07-05 18:09:03 +02:00

Pffff… this has been a nightmare to understand fully, and incoming fix only partially addresses the issue, which basically was that this code was not designed to handle recursive instancing at all.

The incoming commit, which extends/pushes a bit forward current code, is going to add some minimal support for recursive parenting (the case reported here will be working as expected e.g.), however it wont work fully in all complex cases. That would require some refactor, which is out of scope of this task, and not to be done before 2.80 is released anyway.

Pffff… this has been a nightmare to understand fully, and incoming fix only partially addresses the issue, which basically was that this code was not designed to handle recursive instancing at all. The incoming commit, which extends/pushes a bit forward current code, is going to add some minimal support for recursive parenting (the case reported here will be working as expected e.g.), however it wont work fully in all complex cases. That would require some refactor, which is out of scope of this task, and not to be done before 2.80 is released anyway.

blender-admin commented

2019-07-05 18:09:17 +02:00

This issue was referenced by 699193713f

This issue was referenced by 699193713fc003a51d8b0c717856e3543f3a4ee5

Bastien Montagne commented

2019-07-05 18:38:28 +02:00

Changed status from 'Open' to: 'Resolved'

Bastien Montagne closed this issue

2019-07-05 18:38:28 +02:00

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

Make Instances Real: Keep Hierarchy option has no effect when duplicates are collection instances #66282