dyntopo crash #120400

New Issue

ssoneme · 2024-04-08T14:22:38+02:00

ssoneme commented

2024-04-08 14:22:38 +02:00

System Information
Operating system: win 11 pro version 10.0.22631 build 22631
Graphics card: rtx 2060 super / ryzen 5 3600

Blender Version
Blender Version: 4.1.1

Short description of error
dyntopo cause crash when enabling

>	blender.exe!BKE_pbvh_type(const PBVH * pbvh) Line 79	C++
 	blender.exe!sculpt_attr_update(Object * ob, SculptAttribute * attr) Line 2492	C++
 	blender.exe!sculpt_attribute_update_refs(Object * ob) Line 2704	C++
 	blender.exe!sculpt_attribute_ensure_ex(Object * ob, blender::bke::AttrDomain domain, eCustomDataType proptype, const char * name, const SculptAttributeParams * params, PBVHType pbvhtype, bool flat_array_for_bmesh) Line 2626	C++
 	blender.exe!sculptsession_bmesh_add_layers(Object * ob) Line 2660	C++
 	blender.exe!blender::bke::build_pbvh_for_dynamic_topology(Object * ob) Line 2116	C++
 	blender.exe!BKE_sculpt_object_pbvh_ensure(Depsgraph * depsgraph, Object * ob) Line 2186	C++
 	blender.exe!sculpt_update_object(Depsgraph * depsgraph, Object * ob, Object * ob_eval, bool is_paint_tool) Line 1742	C++
 	blender.exe!BKE_sculpt_update_object_after_eval(Depsgraph * depsgraph, Object * ob_eval) Line 1896	C++
 	blender.exe!mesh_build_data(Depsgraph * depsgraph, const Scene * scene, Object * ob, const CustomData_MeshMasks * dataMask, const bool need_mapping) Line 1339	C++
 	blender.exe!makeDerivedMesh(Depsgraph * depsgraph, const Scene * scene, Object * ob, const CustomData_MeshMasks * dataMask) Line 1475	C++
 	blender.exe!BKE_object_handle_data_update(Depsgraph * depsgraph, Scene * scene, Object * ob) Line 173	C++
 	blender.exe!BKE_object_eval_uber_data(Depsgraph * depsgraph, Scene * scene, Object * ob) Line 328	C++
 	blender.exe!blender::deg::DepsgraphNodeBuilder::build_object_data_geometry::__l2::<lambda>(Depsgraph * depsgraph) Line 1661	C++
 	[External Code]	
 	blender.exe!blender::deg::`anonymous namespace'::evaluate_node(const blender::deg::`anonymous-namespace'::DepsgraphEvalState * state, blender::deg::OperationNode * operation_node) Line 108	C++
 	blender.exe!blender::deg::`anonymous namespace'::deg_task_run_func(TaskPool * pool, void * taskdata) Line 121	C++
 	blender.exe!Task::operator()() Line 167	C++
 	blender.exe!tbb::internal::function_task<Task>::execute() Line 1060	C++
 	[External Code]

Exact steps for others to reproduce the error

open attached .blend file sculpting_body_female_23.blend
Grab and move some point in the ear
Enable Dyntopo

**System Information** Operating system: win 11 pro version 10.0.22631 build 22631 Graphics card: rtx 2060 super / ryzen 5 3600 **Blender Version** Blender Version: 4.1.1 **Short description of error** dyntopo cause crash when enabling ``` > blender.exe!BKE_pbvh_type(const PBVH * pbvh) Line 79 C++ blender.exe!sculpt_attr_update(Object * ob, SculptAttribute * attr) Line 2492 C++ blender.exe!sculpt_attribute_update_refs(Object * ob) Line 2704 C++ blender.exe!sculpt_attribute_ensure_ex(Object * ob, blender::bke::AttrDomain domain, eCustomDataType proptype, const char * name, const SculptAttributeParams * params, PBVHType pbvhtype, bool flat_array_for_bmesh) Line 2626 C++ blender.exe!sculptsession_bmesh_add_layers(Object * ob) Line 2660 C++ blender.exe!blender::bke::build_pbvh_for_dynamic_topology(Object * ob) Line 2116 C++ blender.exe!BKE_sculpt_object_pbvh_ensure(Depsgraph * depsgraph, Object * ob) Line 2186 C++ blender.exe!sculpt_update_object(Depsgraph * depsgraph, Object * ob, Object * ob_eval, bool is_paint_tool) Line 1742 C++ blender.exe!BKE_sculpt_update_object_after_eval(Depsgraph * depsgraph, Object * ob_eval) Line 1896 C++ blender.exe!mesh_build_data(Depsgraph * depsgraph, const Scene * scene, Object * ob, const CustomData_MeshMasks * dataMask, const bool need_mapping) Line 1339 C++ blender.exe!makeDerivedMesh(Depsgraph * depsgraph, const Scene * scene, Object * ob, const CustomData_MeshMasks * dataMask) Line 1475 C++ blender.exe!BKE_object_handle_data_update(Depsgraph * depsgraph, Scene * scene, Object * ob) Line 173 C++ blender.exe!BKE_object_eval_uber_data(Depsgraph * depsgraph, Scene * scene, Object * ob) Line 328 C++ blender.exe!blender::deg::DepsgraphNodeBuilder::build_object_data_geometry::__l2::<lambda>(Depsgraph * depsgraph) Line 1661 C++ [External Code] blender.exe!blender::deg::`anonymous namespace'::evaluate_node(const blender::deg::`anonymous-namespace'::DepsgraphEvalState * state, blender::deg::OperationNode * operation_node) Line 108 C++ blender.exe!blender::deg::`anonymous namespace'::deg_task_run_func(TaskPool * pool, void * taskdata) Line 121 C++ blender.exe!Task::operator()() Line 167 C++ blender.exe!tbb::internal::function_task<Task>::execute() Line 1060 C++ [External Code] ``` **Exact steps for others to reproduce the error** - open attached .blend file [sculpting_body_female_23.blend](attachments/31e0b33d-5be1-48ff-868d-8a33432839ca) - Grab and move some point in the ear - Enable Dyntopo

blender.png

22 KiB

sculpt_dyntopo.crash.txt

53 KiB

ssoneme added the

labels 2024-04-08 14:22:39 +02:00

Germano Cavalcante commented

2024-04-08 20:36:32 +02:00

Thank you for the report.
To help us investigate the issue, could you please provide more details on how to replicate the crash?

Does the crash happen with any mesh, or is it specific to a particular mesh?
If it is specific to a certain mesh, could you provide the mesh for further testing?
Have you tested if the issue occurs in older versions of Blender?
Additionally, could you try the latest daily build of Blender to see if the issue persists? You can download it from this link: https://builder.blender.org/download/

Thank you for the report. To help us investigate the issue, could you please provide more details on how to replicate the crash? - Does the crash happen with any mesh, or is it specific to a particular mesh? - If it is specific to a certain mesh, could you provide the mesh for further testing? - Have you tested if the issue occurs in older versions of Blender? - Additionally, could you try the latest daily build of Blender to see if the issue persists? You can download it from this link: https://builder.blender.org/download/

Germano Cavalcante added

Status

Needs Information from User

and removed

Status

Needs Triage

labels 2024-04-08 20:36:43 +02:00

ssoneme commented

2024-04-09 01:58:10 +02:00

Haven't tested in other projects as I'm only doing one. The last time it crashed was when I was sculpting ears and switched to dyntopo and it doesn't happen all the time, every once in a while. I am using blender via steam btw.

sculpting_body_female_23.blend

9.4 MiB

Germano Cavalcante commented

2024-04-09 14:07:48 +02:00

We had some corrections in the sculpt area.
It would be nice if you could check if this problem has already been potentially fixed in version 4.1.1
https://builder.blender.org/download/

Steam and other platforms have not yet made this version available.

We had some corrections in the sculpt area. It would be nice if you could check if this problem has already been potentially fixed in version 4.1.1 https://builder.blender.org/download/ Steam and other platforms have not yet made this version available.

ssoneme commented

2024-04-09 15:38:23 +02:00

Checked in version 4.1.1 also crashes.

Germano Cavalcante added

Module

Sculpt, Paint & Texture

and removed

Needs Information from User

labels 2024-04-09 17:31:54 +02:00

Germano Cavalcante commented

2024-04-09 17:33:49 +02:00

Thank you for testing the issue.
I was able to reproduce the crash on Blender 4.1.1 and added steps for others to reproduce the issue.
I can confirm the issue and will forward it to the developers for further investigation.

Thank you for testing the issue. I was able to reproduce the crash on Blender 4.1.1 and added steps for others to reproduce the issue. I can confirm the issue and will forward it to the developers for further investigation.

ssoneme commented

2024-04-09 18:17:28 +02:00

thanks

ssoneme closed this issue

2024-04-09 18:17:31 +02:00

Blender Bot added

and removed

labels 2024-04-09 18:17:35 +02:00

Germano Cavalcante reopened this issue

2024-04-09 18:24:10 +02:00

Blender Bot added

and removed

labels 2024-04-09 18:24:13 +02:00

Germano Cavalcante added

and removed

labels 2024-04-09 18:24:23 +02:00

Germano Cavalcante commented

2024-04-09 19:59:43 +02:00

Caused by b3aca5b28f
@HooglyBoogly ^

Caused by b3aca5b28f @HooglyBoogly ^

Sean Kim commented

2024-04-09 22:34:47 +02:00

Bug is this line inside sculpt_attr_update:

  /* Check if we are a coerced simple array and shouldn't be. */
  bad |= attr->simple_array && !attr->params.simple_array &&
         !ELEM(BKE_pbvh_type(ss->pbvh), PBVH_GRIDS, PBVH_BMESH);

Caused by the change that @mano-wii pointed out in build_pbvh_for_dynamic_topology where sculptsession_bmesh_add_layers(ob); expects a PBVH to exist in a later function call.

~~Should have a fix for this shortly~~

Edit: Simple fix isn't so simple due to some interdependencies in this area, will take a bit longer to make a fix for this

Bug is this line inside `sculpt_attr_update`: ``` /* Check if we are a coerced simple array and shouldn't be. */ bad |= attr->simple_array && !attr->params.simple_array && !ELEM(BKE_pbvh_type(ss->pbvh), PBVH_GRIDS, PBVH_BMESH); ``` Caused by the change that @mano-wii pointed out in `build_pbvh_for_dynamic_topology` where `sculptsession_bmesh_add_layers(ob);` expects a PBVH to exist in a later function call. ~~Should have a fix for this shortly~~ Edit: Simple fix isn't so simple due to some interdependencies in this area, will take a bit longer to make a fix for this

Sean Kim referenced a pull request that will close this issue

2024-04-10 00:08:56 +02:00

Fix #120400: Remove cyclic dependency of pbvh type when building bmesh pbvh #120445

Hans Goudey added

and removed

labels 2024-04-10 02:20:44 +02:00

Hans Goudey referenced this issue from a commit

2024-04-10 02:57:45 +02:00