Active Color Attribute switches after save and reverting file #107067

New Issue

Julien Kaspar · 2023-04-18T11:56:53+02:00

Julien Kaspar commented

2023-04-18 11:56:53 +02:00

System Information
Operating system: Linux-6.2.11-gentoo-x86_64-AMD_Ryzen_Threadripper_1950X_16-Core_Processor-with-glibc2.36 64 Bits, X11 UI
Graphics card: Quadro GP100/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 530.41.03

Blender Version
Broken: version: 3.6.0 Alpha, branch: HEAD (modified), commit date: 2023-04-16 10:41, hash: e1571cb10517

Caused by 6514bb05ea

Short description of error
When the last action on an object was to switch between active color attributes, it will switch back to the previously active color attribute after saving and reloading the file.
This can happen on multiple objects at once when reloading. If the color attribute was painted in before saving, it will stay active when reloading.

Exact steps for others to reproduce the error

Add three color attributes in the object data properties on a primitive object (each a different color)
Click on one of them to switch the active one
Save and reload the file
Try this multiple times between different color attributes (each time it will switch back to the previously active one)

Same is true for the default color attribute btw. [Camera icon]

**System Information** Operating system: Linux-6.2.11-gentoo-x86_64-AMD_Ryzen_Threadripper_1950X_16-Core_Processor-with-glibc2.36 64 Bits, X11 UI Graphics card: Quadro GP100/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 530.41.03 **Blender Version** Broken: version: 3.6.0 Alpha, branch: HEAD (modified), commit date: 2023-04-16 10:41, hash: `e1571cb10517` Caused by 6514bb05ea5a **Short description of error** When the last action on an object was to switch between active color attributes, it will switch back to the previously active color attribute after saving and reloading the file. This can happen on multiple objects at once when reloading. If the color attribute was painted in before saving, it will stay active when reloading. **Exact steps for others to reproduce the error** - Add three color attributes in the object data properties on a primitive object (each a different color) - Click on one of them to switch the active one - Save and reload the file - Try this multiple times between different color attributes (each time it will switch back to the previously active one) Same is true for the default color attribute btw. [Camera icon]

Julien Kaspar added the

labels 2023-04-18 11:56:53 +02:00

Philipp Oeser commented

2023-04-19 13:02:04 +02:00

Confirmed

Philipp Oeser added

and removed

labels 2023-04-19 13:02:24 +02:00

Philipp Oeser commented

2023-04-19 13:08:23 +02:00

Same is true for the default color attribute btw. [Camera icon]

Iliya Katushenock commented

2023-04-19 13:10:02 +02:00

There seems to be a lot of issues with active status right now. Looks like @Baardaap was doing something, and now there's a PR associated with it. Not sure if they will help, but it seems close

Philipp Oeser commented

2023-04-19 13:10:35 +02:00

If the color attribute was painted in before saving, it will stay active when reloading.

Not even this is working on my side tbh

> If the color attribute was painted in before saving, it will stay active when reloading. Not even this is working on my side tbh

Philipp Oeser commented

2023-04-19 13:31:04 +02:00

When checking mesh_blend_write, the active_color_attribute is actually there when encoding undo steps (coming from BLO_write_file_mem), but NOT there (NULL), when actually saving the file (coming from BLO_write_file).

Maybe that gives a hint already

When checking `mesh_blend_write`, the active_color_attribute is actually there when encoding undo steps (coming from `BLO_write_file_mem`), but NOT there (NULL), when actually saving the file (coming from `BLO_write_file`). Maybe that gives a hint already

Philipp Oeser commented

2023-04-19 13:54:09 +02:00

Caused by 6514bb05ea

Caused by 6514bb05ea5a

Philipp Oeser added

and removed

labels 2023-04-19 13:54:34 +02:00

Martijn Versteegh commented

2023-04-19 17:19:21 +02:00

That seems related, yes. probably the string doesn't make it into the blendfile?

I'll have a look , but @HooglyBoogly is probably faster in fixing this

That seems related, yes. probably the string doesn't make it into the blendfile? I'll have a look , but @HooglyBoogly is probably faster in fixing this

Martijn Versteegh self-assigned this 2023-04-19 17:48:03 +02:00

Martijn Versteegh referenced a pull request that will close this issue

2023-04-19 21:57:44 +02:00

Fix #107067: Properly clear CD_FLAG_ACTIVE/DEFAULT_COLOR flags #107146

Martijn Versteegh referenced this issue from a commit

2023-04-19 22:27:14 +02:00

Fix #107067: Properly clear CD_FLAG_ACTIVE/DEFAULT_COLOR flags

Martijn Versteegh closed this issue

2023-04-19 22:27:14 +02:00

Blender Bot added

Status

Resolved