Regression: Blender crash when baking Multires displacements #98546

New Issue

James Guilford · 2022-06-01T23:09:37+02:00

James Guilford commented

2022-06-01 23:09:37 +02:00

System Information
Operating system: Windows-10-10.0.22000-SP0 64 Bits
Graphics card: NVIDIA GeForce GTX 1080/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 511.65

Blender Version
Broken: version: 3.1.2, branch: master, commit date: 2022-03-31 17:40, hash: cc66d1020c

Caused by cfa53e0fbe
NOTE: to build cfa53e0fbe, I needed 348631cffe on top

Short description of error
When baking mulitres displacements, if low resolution mesh is checked, blender will crash

Exact steps for others to reproduce the error

Add multires modifier with a few subdivisions to cube or other object
Set multires to 0 in viewport
Set up a texture in shader
In bake settings, Select bake from multires, and bake type ->displacement
In output dropdown, Select 'Low Resolution Mesh'
Click bake -> Crash

Test File with all steps included:
#98546.blend

**System Information** Operating system: Windows-10-10.0.22000-SP0 64 Bits Graphics card: NVIDIA GeForce GTX 1080/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 511.65 **Blender Version** Broken: version: 3.1.2, branch: master, commit date: 2022-03-31 17:40, hash: `cc66d1020c` Caused by cfa53e0fbe NOTE: to build cfa53e0fbe, I needed 348631cffe on top **Short description of error** When baking mulitres displacements, if low resolution mesh is checked, blender will crash **Exact steps for others to reproduce the error** - Add multires modifier with a few subdivisions to cube or other object - Set multires to 0 in viewport - Set up a texture in shader - In bake settings, Select bake from multires, and bake type ->displacement - In output dropdown, Select 'Low Resolution Mesh' - Click bake -> Crash Test File with all steps included: [#98546.blend](https://archive.blender.org/developer/F13128661/T98546.blend)

James Guilford commented

2022-06-01 23:09:37 +02:00

Added subscriber: @fjg3d

Pratik Borhade commented

2022-06-02 07:13:00 +02:00

Added subscriber: @PratikPB2123

Pratik Borhade commented

2022-06-02 07:22:41 +02:00

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

Pratik Borhade commented

2022-06-02 07:22:41 +02:00

Added subscriber: @lichtwerk

Pratik Borhade commented

2022-06-02 07:22:41 +02:00

Hi, thanks for the report. This appears to be a regressions (doesn't crash in 3.0.1)
Will raise the priority.

@lichtwerk , could you bisect?

Hi, thanks for the report. This appears to be a regressions (doesn't crash in 3.0.1) Will raise the priority. @lichtwerk , could you bisect?

Pratik Borhade changed title from ~~Blender crash when baking Multires displacements (with 'Low Resolution Mesh' checked)~~ to Regression: Blender crash when baking Multires displacements

2022-06-02 07:25:38 +02:00

Pratik Borhade commented

2022-06-02 07:34:40 +02:00

hmm maybe {cfa53e0fbe}?
Broke between 97c2c3991620 - e69020ad0f52

hmm maybe {cfa53e0fbe}? Broke between `97c2c3991620` - `e69020ad0f52`

Philipp Oeser commented

2022-06-02 09:20:35 +02:00

Will check

Philipp Oeser commented

2022-06-02 11:30:53 +02:00

Added subscribers: @HooglyBoogly, @brecht

Philipp Oeser commented

2022-06-02 11:30:53 +02:00

Caused by cfa53e0fbe indeed.

NOTE: to build cfa53e0fbe, I needed 348631cffe on top

Only crashes with multires viewport levels set 0, so there might be something wrong with just using the orig DerivedMesh in multires_make_derived_from_derived / multiresbake_create_loresdm?
We seem to be missing vert_normals on the CDDerivedMesh.
Precomputed normals are still available on the MResolvePixelData in flush_pixel, maybe they could be used also?
Or just the ususal BKE_mesh_XXX_normals_ensure missing somewhere in the process?

1  copy_v3_v3              math_vector_inline.c 44  0x463d777      
2  cdDM_getVertNo          cdderivedmesh.c      112 0x463d9cf      
3  interp_bilinear_mpoly   multires_bake.c      727 0x61c2fd7      
4  apply_heights_callback  multires_bake.c      879 0x61c38ef      
5  flush_pixel             multires_bake.c      199 0x61c0cd2      
6  set_rast_triangle       multires_bake.c      220 0x61c0d7a      
7  rasterize_half          multires_bake.c      272 0x61c0fa3      
8  bake_rasterize          multires_bake.c      321 0x61c144b      
9  do_multires_bake_thread multires_bake.c      405 0x61c16c7      
10 tslot_thread_start      threads.cc           204 0x121bc959     
11 start_thread                                     0x7ffff7a8f0e7 
12 clone3                                           0x7ffff7b14780

@HooglyBoogly , @brecht : is a fix still doable for 3.2?

Caused by cfa53e0fbe indeed. NOTE: to build cfa53e0fbe, I needed 348631cffe on top Only crashes with multires viewport levels set 0, so there might be something wrong with just using the orig `DerivedMesh` in `multires_make_derived_from_derived` / `multiresbake_create_loresdm`? We seem to be missing `vert_normals` on the `CDDerivedMesh`. Precomputed normals are still available on the `MResolvePixelData` in `flush_pixel`, maybe they could be used also? Or just the ususal `BKE_mesh_XXX_normals_ensure` missing somewhere in the process? ``` 1 copy_v3_v3 math_vector_inline.c 44 0x463d777 2 cdDM_getVertNo cdderivedmesh.c 112 0x463d9cf 3 interp_bilinear_mpoly multires_bake.c 727 0x61c2fd7 4 apply_heights_callback multires_bake.c 879 0x61c38ef 5 flush_pixel multires_bake.c 199 0x61c0cd2 6 set_rast_triangle multires_bake.c 220 0x61c0d7a 7 rasterize_half multires_bake.c 272 0x61c0fa3 8 bake_rasterize multires_bake.c 321 0x61c144b 9 do_multires_bake_thread multires_bake.c 405 0x61c16c7 10 tslot_thread_start threads.cc 204 0x121bc959 11 start_thread 0x7ffff7a8f0e7 12 clone3 0x7ffff7b14780 ``` @HooglyBoogly , @brecht : is a fix still doable for 3.2?

Hans Goudey self-assigned this 2022-06-02 12:52:11 +02:00

James Guilford commented

2022-06-02 17:32:09 +02:00

BTW, is the low resolution mesh option the same as manually setting the multires to level 0 in the modifier?

blender-admin commented

2022-06-02 17:46:38 +02:00

This issue was referenced by 96a47af413

This issue was referenced by 96a47af413a8055f50b9989905922588494d1653

Hans Goudey commented

2022-06-02 17:47:11 +02:00

Changed status from 'Confirmed' to: 'Resolved'

Hans Goudey closed this issue

2022-06-02 17:47:11 +02:00

Thomas Dinges added this to the 3.2 milestone 2023-02-08 15:51:07 +01: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

Regression: Blender crash when baking Multires displacements #98546