Multires: Subdivision Set Relative is subdividing the object #110879

New Issue

Julien Kaspar · 2023-08-07T10:28:13+02:00

Julien Kaspar commented

2023-08-07 10:28:13 +02:00

System Information
Operating system: Linux-6.4.6-gentoo-x86_64-AMD_Ryzen_Threadripper_1950X_16-Core_Processor-with-glibc2.37 64 Bits, X11 UI
Graphics card: AMD Radeon RX 7900 XT (gfx1100, LLVM 15.0.7, DRM 3.52, 6.4.6-gentoo) AMD 4.6 (Core Profile) Mesa 23.3.0-devel (git-50c29e1ffa)

Blender Version
Broken: version: 4.0.0 Alpha, branch: main, commit date: 2023-08-06 21:11, hash: 93728fee53dc
Worked: --

Short description of error
With 6de6d7267f the shortcuts to increase or decrease the current multires level in sculpt mode are more accessible and more often used. This highlighted an issue.

There is a corner case where the relative level change is subdividing the multires.
If there is no multires detected yet, object.subdivision_set will create a multires modifier and the target level. This is useful when using the shortcuts Ctrl 1-6 to set an absolute level.
But the purpose of the relative option is just to go up & down existing levels.
When set to relative, it is not able to create new levels when a multires modifier is already detected.

This is an especially bad issue in the industry compatible keymap, where the shortcuts are currently being changed to be more accessible on D which is then very easy to press by accident and subdivide the mesh (#109750).

The ideal behavior would be that the object.subdivision_set, if set to relative, cannot be able to subdivide nor create a new multires modifier. It can only change existing levels on the an existing multires/subdiv modifier.

Exact steps for others to reproduce the error

Create a new object
Enter sculpt mode
Press Alt 2
A new multires modifier has been created with a single subdivision
Press Alt 1 and Alt 2 to lower and increase the level
Note how no new levels are created

**System Information** Operating system: Linux-6.4.6-gentoo-x86_64-AMD_Ryzen_Threadripper_1950X_16-Core_Processor-with-glibc2.37 64 Bits, X11 UI Graphics card: AMD Radeon RX 7900 XT (gfx1100, LLVM 15.0.7, DRM 3.52, 6.4.6-gentoo) AMD 4.6 (Core Profile) Mesa 23.3.0-devel (git-50c29e1ffa) **Blender Version** Broken: version: 4.0.0 Alpha, branch: main, commit date: 2023-08-06 21:11, hash: `93728fee53dc` Worked: -- **Short description of error** With 6de6d7267f the shortcuts to increase or decrease the current multires level in sculpt mode are more accessible and more often used. This highlighted an issue. There is a corner case where the relative level change is subdividing the multires. If there is no multires detected yet, `object.subdivision_set` will create a multires modifier and the target level. This is useful when using the shortcuts `Ctrl 1-6` to set an absolute level. But the purpose of the relative option is just to go up & down existing levels. When set to relative, it is not able to create new levels when a multires modifier is already detected. This is an especially bad issue in the industry compatible keymap, where the shortcuts are currently being changed to be more accessible on `D` which is then very easy to press by accident and subdivide the mesh (#109750). The ideal behavior would be that the `object.subdivision_set`, if set to relative, cannot be able to subdivide nor create a new multires modifier. It can only change existing levels on the an existing multires/subdiv modifier. **Exact steps for others to reproduce the error** - Create a new object - Enter sculpt mode - Press `Alt 2` - A new multires modifier has been created with a single subdivision - Press `Alt 1` and `Alt 2` to lower and increase the level - Note how no new levels are created

❤️ 1

Julien Kaspar added the

Sculpt, Paint & Texture

labels 2023-08-07 10:29:01 +02:00

Iliya Katushenock added