Maniphest T101053

Regression: Shade smooth operator disabling auto smooth
Closed, Archived
Actions

Assigned To
None
Authored By
Konstantin (miller)
Sep 14 2022, 7:01 AM
Tags
  • BF Blender
Subscribers
EP (impacman)
Konstantin (miller)
Omar Emara (OmarSquircleArt)

Description

System Information
Operating system: Windows-10-10.0.19044-SP0 64 Bits
Graphics card: NVIDIA GeForce RTX 2060/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 512.59

Blender Version
Broken: version: 3.3.0, branch: master, commit date: 2022-09-06 15:39, hash: rB0759f671ce1f
Worked: (newest version of Blender that worked as expected)

Short description of error
bpy.ops.object.shade_smooth() also disabling Auto Smooth (bpy.context.object.data.use_auto_smooth)

Exact steps for others to reproduce the error

  1. Open Blender
  2. Add Suzane
  3. Enable Auto Smooth in object properties
  4. Set smooth shading on Suzane(right click/w - Shade Smooth)

Related Objects

Event Timeline

Konstantin (miller) created this task.Sep 14 2022, 7:01 AM
Konstantin (miller) updated the task description.Sep 14 2022, 7:05 AM
Konstantin (miller) added a comment.Sep 14 2022, 8:24 AM

If we have this new operator where we can adjust paraments


Why we have two separates buttons for one action.
It's inconsistent with the rest.
For example by this logic we should add different subdivision levels in right click menu like
Subdivide x2 Subdivide x3 etc...

Omar Emara (OmarSquircleArt) closed this task as Archived.Sep 14 2022, 9:53 AM
Omar Emara (OmarSquircleArt) added a subscriber: Omar Emara (OmarSquircleArt).

This was intentionally added in D14894, so it is not a bug.

So the issue reported here is a request for modified/improved behavior and not a bug in current behavior. Closing as this bug tracker is only for bugs and errors.

For user requests and feedback, please use other channels: https://wiki.blender.org/wiki/Communication/Contact#User_Feedback_and_Requests

For more information on why this isn't considered a bug, visit: https://wiki.blender.org/wiki/Reference/Not_a_bug

Konstantin (miller) added a comment.Sep 14 2022, 10:23 AM

@Omar Emara (OmarSquircleArt)
Sure, wasn't aware about that. I spot it because my scripts were acting weird on 3.3...
But this is doesn't mean I'm incorrect with my comment statement and the new feature completely got think trough
Right now Sade Smooth doing same as Shade Auto Smooth but removing Auto Smooth from obj...
But if you adjust last operation with |Auto Smooth = True| after Shade Smooth you end up with two buttons doing same

Konstantin (miller) mentioned this in D14894: Mesh: Add Auto Smooth option to Shade Smooth operator.Sep 14 2022, 10:41 AM
EP (impacman) added a subscriber: EP (impacman).Oct 18 2022, 2:32 PM
In T101053#1416880, @Omar Emara (OmarSquircleArt) wrote:

This was intentionally added in D14894, so it is not a bug.

So the issue reported here is a request for modified/improved behavior and not a bug in current behavior. Closing as this bug tracker is only for bugs and errors.

For user requests and feedback, please use other channels: https://wiki.blender.org/wiki/Communication/Contact#User_Feedback_and_Requests

For more information on why this isn't considered a bug, visit: https://wiki.blender.org/wiki/Reference/Not_a_bug

Miller talk about not the new option as bug but about two exact same operators in right click menu "shade smooth" and "auto shade smooth". Its duplication