Regression: Missing default face set for primitive meshes #101722

New Issue

Pratik Borhade · 2022-10-10T12:56:27+02:00

Pratik Borhade commented

2022-10-10 12:56:27 +02:00

System Information
Operating system: Windows-10-10.0.22000-SP0 64 Bits
Graphics card: NVIDIA GeForce RTX 3050 Laptop GPU/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 512.78

Blender Version
Broken: builds before 22c3db72ca or apply D16206 on master
Worked: 3.3
First bad commit: {b5f7af31d6d474c3b455bacda079969fde7c0962}

Short description of error
I guess default face set for primitive meshes are missing.
Because calling Hide active face sets doesn't hide the mesh. It works though for Default cube

Exact steps for others to reproduce the error

Open default scene
delete cube and add new mesh
Switch to sculpt mode
Press {key Shift H} to hide all face sets

Found this issue while bisecting #101699 (Regression: Crash in sculpt mode using shift+h )

**System Information** Operating system: Windows-10-10.0.22000-SP0 64 Bits Graphics card: NVIDIA GeForce RTX 3050 Laptop GPU/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 512.78 **Blender Version** Broken: builds before 22c3db72ca or apply [D16206](https://archive.blender.org/developer/D16206) on master Worked: 3.3 First bad commit: {b5f7af31d6d474c3b455bacda079969fde7c0962} **Short description of error** I guess default face set for primitive meshes are missing. Because calling `Hide active face sets` doesn't hide the mesh. It works though for `Default cube` **Exact steps for others to reproduce the error** - Open default scene - delete cube and add new mesh - Switch to sculpt mode - Press {key Shift H} to hide all face sets --- Found this issue while bisecting #101699 (Regression: Crash in sculpt mode using shift+h )

Pratik Borhade commented

2022-10-10 12:56:27 +02:00

Added subscriber: @PratikPB2123

Pratik Borhade commented

2022-10-10 12:56:38 +02:00

Added subscriber: @HooglyBoogly

Omar Emara commented

2022-10-10 16:01:33 +02:00

Added subscriber: @OmarEmaraDev

Omar Emara commented

2022-10-10 16:01:33 +02:00

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

Omar Emara commented

2022-10-10 16:01:33 +02:00

I hit an assert as well:

BLI_assert failed: source/blender/editors/sculpt_paint/sculpt.c:372, SCULPT_face_set_visibility_set(), at 'ss->face_sets != ((void*)0)'

I hit an assert as well: ``` BLI_assert failed: source/blender/editors/sculpt_paint/sculpt.c:372, SCULPT_face_set_visibility_set(), at 'ss->face_sets != ((void*)0)' ```

Pratik Borhade commented

2022-10-10 16:29:14 +02:00

In #101722#1430215, @OmarEmaraDev wrote:
I hit an assert as well:
BLI_assert failed: source/blender/editors/sculpt_paint/sculpt.c:372, SCULPT_face_set_visibility_set(), at 'ss->face_sets != ((void*)0)'

yes, thanks different issue :)
see: #101699 (Regression: Crash in sculpt mode using shift+h )

> In #101722#1430215, @OmarEmaraDev wrote: > I hit an assert as well: > ``` > BLI_assert failed: source/blender/editors/sculpt_paint/sculpt.c:372, SCULPT_face_set_visibility_set(), at 'ss->face_sets != ((void*)0)' > ``` yes, thanks different issue :) see: #101699 (Regression: Crash in sculpt mode using shift+h )

Hans Goudey self-assigned this 2022-10-10 17:14:06 +02:00

TheRedWaxPolice commented

2022-10-11 03:59:40 +02:00

Added subscriber: @TheRedWaxPolice

Pratik Borhade changed title from ~~Missing default face set for primitive meshes~~ to Regression: Missing default face set for primitive meshes

2022-10-11 10:59:01 +02:00

Pratik Borhade commented

2022-10-11 11:08:55 +02:00

Unable to redo this anymore in current master. Can someone confirm?
Probably fixed by da25006bc4

Unable to redo this anymore in current master. Can someone confirm? Probably fixed by da25006bc4

Omar Emara commented

2022-10-11 11:25:47 +02:00

Can't redo on latest master either.