Regression: Merging objects does not assign materials correctly #100822

New Issue

RedHaloStudio · 2022-09-05T04:21:42+02:00

RedHaloStudio commented

2022-09-05 04:21:42 +02:00

System Information
Operating system: Windows-10-10.0.19041-SP0 64 Bits
Graphics card: NVIDIA GeForce RTX 3090/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 516.94

Blender Version
Broken: version: 3.4.0 Alpha, branch: master, commit date: 2022-09-03 09:43, hash: 4b818b1513
Worked: (newest version of Blender that worked as expected)
Caused by f1c0249f34

Short description of error
A object has two materials: RED BLUE(in order)
B object has two materials: GREEN RED(in order)
When A and B are merged, the material is incorrectly specified

blender file:
BUG.blend

Exact steps for others to reproduce the error
[Please describe the exact steps needed to reproduce the issue]
[Based on the default startup or an attached .blend file (as simple as possible)]

**System Information** Operating system: Windows-10-10.0.19041-SP0 64 Bits Graphics card: NVIDIA GeForce RTX 3090/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 516.94 **Blender Version** Broken: version: 3.4.0 Alpha, branch: master, commit date: 2022-09-03 09:43, hash: `4b818b1513` Worked: (newest version of Blender that worked as expected) Caused by f1c0249f34 **Short description of error** A object has two materials: RED BLUE(in order) B object has two materials: GREEN RED(in order) When A and B are merged, the material is incorrectly specified ![QQ截图20220905101106.jpg](https://archive.blender.org/developer/F13460405/QQ截图20220905101106.jpg) blender file: [BUG.blend](https://archive.blender.org/developer/F13460407/BUG.blend) **Exact steps for others to reproduce the error** [Please describe the exact steps needed to reproduce the issue] [Based on the default startup or an attached .blend file (as simple as possible)]

RedHaloStudio commented

2022-09-05 04:21:42 +02:00

Added subscriber: @HMRP

blender-admin commented

2022-09-05 04:21:42 +02:00

blender/blender-addons#100839 was marked as duplicate of this issue

Pratik Borhade commented

2022-09-05 05:24:58 +02:00

Added subscriber: @PratikPB2123

Pratik Borhade commented

2022-09-05 05:24:58 +02:00

Changed status from 'Needs Triage' to: 'Confirmed'

Pratik Borhade commented

2022-09-05 05:24:58 +02:00

Hi, thanks for the report. Can redo this in latest master build. This does not happen in previous release versions.
Will find when exactly this changed

Hi, thanks for the report. Can redo this in latest master build. This does not happen in previous release versions. Will find when exactly this changed

Pratik Borhade commented

2022-09-05 06:33:15 +02:00

Added subscriber: @HooglyBoogly

Pratik Borhade commented

2022-09-05 06:33:15 +02:00

Caused by f1c0249f34
@HooglyBoogly ^

Caused by f1c0249f34 @HooglyBoogly ^

Pratik Borhade changed title from ~~Merging objects does not assign materials correctly~~ to Regression: Merging objects does not assign materials correctly

2022-09-05 06:33:45 +02:00

Philipp Oeser commented

2022-09-05 10:54:20 +02:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2022-09-05 10:54:20 +02:00

Looked at this a bit, couple of things I noticed in join_mesh_single:

shouldnt the AttributeWriter be on the destination mesh?
shouldnt this take into account the polyofs?
the size of the MutableVArraySpan is also not the size of the combined mesh?

Looked at this a bit, couple of things I noticed in `join_mesh_single`: - shouldnt the `AttributeWriter` be on the destination mesh? - shouldnt this take into account the `polyofs`? - the size of the MutableVArraySpan is also not the size of the combined mesh?

Philipp Oeser commented

2022-09-05 12:17:56 +02:00

Think I got a fix which does the following:

work on the destination polydata
take polyofs into account

(bit on shaky ground, but will post nonetheless)

Think I got a fix which does the following: - work on the destination polydata - take polyofs into account (bit on shaky ground, but will post nonetheless)

blender-admin commented

2022-09-05 12:22:51 +02:00

This issue was referenced by ecf3287533

This issue was referenced by ecf3287533c8adc90250bc13957eddb7b2e22fc6

Muruga Jothi commented

2022-09-05 18:55:28 +02:00

Added subscriber: @pm_jothi