Regression: Selection Fails to Highlight in Edit Mode (with restposition and modifier/shapekey displayed in editmode), also crash switching to wireframe or x-ray #109432

Closed
opened 2023-06-28 05:57:38 +02:00 by Chris Jones · 15 comments

System Information
Operating system: Windows-10-10.0.22621-SP0 64 Bits
Graphics card: Intel(R) UHD Graphics 770 Intel 4.5.0 - Build 31.0.101.3959

Blender Version
Broken: version: 3.6.0, branch: blender-v3.6-release, commit date: 2023-06-27 08:08, hash: c7fc78b81ecb
Worked: version: 3.5.1, branch: blender-v3.5-release, commit date: 2023-04-24 18:11, hash: e1ccd9d4a1d3

Caused by 82ca3d3604

Short description of error
When selecting vertices, edges or faces in either of the attached example files, there is no visible indication of the selection in the 3D viewport.

Exact steps for others to reproduce the error

Selection Bug.blend

  1. Enter Edit mode and try to select a vertex, edge or face
  2. Result: UVs become visible in the UV Editor, but nothing happens in the 3D Viewport
  3. Remove the Basis shape key and selection highlighting becomes visible

Selection Bug.blend

  1. Enter Edit mode
  2. Switch to face select mode and then to wireframe or x-ray view to crash Blender

Selection Bug B.blend

  1. Enter Edit mode and try to select a vertex, edge or face
  2. Result: UVs become visible in the UV Editor, but nothing happens in the 3D Viewport
  3. Disable the Warp modifier and selection highlighting becomes visible
**System Information** Operating system: Windows-10-10.0.22621-SP0 64 Bits Graphics card: Intel(R) UHD Graphics 770 Intel 4.5.0 - Build 31.0.101.3959 **Blender Version** Broken: version: 3.6.0, branch: blender-v3.6-release, commit date: 2023-06-27 08:08, hash: `c7fc78b81ecb` Worked: version: 3.5.1, branch: blender-v3.5-release, commit date: 2023-04-24 18:11, hash: `e1ccd9d4a1d3` Caused by 82ca3d3604 **Short description of error** When selecting vertices, edges or faces in either of the attached example files, there is no visible indication of the selection in the 3D viewport. **Exact steps for others to reproduce the error** **Selection Bug.blend** 1. Enter Edit mode and try to select a vertex, edge or face 2. Result: UVs become visible in the UV Editor, but nothing happens in the 3D Viewport 3. Remove the Basis shape key and selection highlighting becomes visible **Selection Bug.blend** 1. Enter Edit mode 2. Switch to face select mode and then to wireframe or x-ray view to crash Blender **Selection Bug B.blend** 1. Enter Edit mode and try to select a vertex, edge or face 2. Result: UVs become visible in the UV Editor, but nothing happens in the 3D Viewport 3. Disable the Warp modifier and selection highlighting becomes visible
Chris Jones added the
Type
Report
Status
Needs Triage
Severity
Normal
labels 2023-06-28 05:57:38 +02:00
Member

Thanks for the report. I can confirm.

Thanks for the report. I can confirm.
Member

broke between bd86e719ab91 - 993d24c1749c

@lichtwerk , can you check further?

broke between `bd86e719ab91 - 993d24c1749c` @lichtwerk , can you check further?
Member

Guess this is related to 82ca3d3604

Guess this is related to 82ca3d3604bded96f72650c23a7f95f29b956f30
Pratik Borhade changed title from Selection Fails to Highlight in Edit Mode to Regression: Selection Fails to Highlight in Edit Mode 2023-06-28 07:14:17 +02:00
Member

Checking...

Checking...
Member

Yep, caused by 82ca3d3604

Yep, caused by 82ca3d3604
Philipp Oeser changed title from Regression: Selection Fails to Highlight in Edit Mode to Regression: Selection Fails to Highlight in Edit Mode (with restposition and modifier/shapekey displayed in editmode) 2023-06-28 10:21:30 +02:00
Member

Thanks :D

Yep, caused by 82ca3d3604

@HooglyBoogly ^

Thanks :D > Yep, caused by 82ca3d3604 @HooglyBoogly ^
Hans Goudey self-assigned this 2023-06-28 19:11:26 +02:00
Member

This will be fixed in 4.0 by #108637. But I'll try to split out the smallest change that fixes this issue too, so it can be back-ported to 3.6.

This will be fixed in 4.0 by #108637. But I'll try to split out the smallest change that fixes this issue too, so it can be back-ported to 3.6.
Member

Actually, I suggest we just revert that commit in 82ca3d3604 or consider this a known issue in 3.6. "Add rest position" is quite a new feature that is only really helpful for the new hair system anyway. I've been trying to split off some smaller changes from #108637 that might fix this, but haven't found anything yet, that change isn't really safe enough to backport.

Actually, I suggest we just revert that commit in 82ca3d3604 or consider this a known issue in 3.6. "Add rest position" is quite a new feature that is only really helpful for the new hair system anyway. I've been trying to split off some smaller changes from #108637 that might fix this, but haven't found anything yet, that change isn't really safe enough to backport.

#109591 the report is also caused by this, but there is also a crash.

https://projects.blender.org/blender/blender/issues/109591 the report is also caused by this, but there is also a crash.
Philipp Oeser changed title from Regression: Selection Fails to Highlight in Edit Mode (with restposition and modifier/shapekey displayed in editmode) to Regression: Selection Fails to Highlight in Edit Mode (with restposition and modifier/shapekey displayed in editmode), also crash switching to wireframe or x-ray 2023-07-03 12:55:31 +02:00
Hans Goudey added this to the 3.6 LTS milestone 2023-07-05 22:55:43 +02:00

I am just curious why fix goes to 4.0, but not in LTS? Isn't that it should be more stable than regular?

I am just curious why fix goes to 4.0, but not in LTS? Isn't that it should be more stable than regular?
Member

The fix is a relatively large refactor that might cause other bugs, so unfortunately it's not quite that straightforward.

The fix is a relatively large refactor that might cause other bugs, so unfortunately it's not quite that straightforward.

But this bug creates stable reproducible crash which is in LTS and only in LTS. Shouldn't #107190 be reverted then?

But this bug creates stable reproducible crash which is in LTS and only in LTS. Shouldn't https://projects.blender.org/blender/blender/pulls/107190 be reverted then?

Actually, I suggest we just revert that commit in 82ca3d3604 or consider this a known issue in 3.6. "Add rest position" is quite a new feature that is only really helpful for the new hair system anyway. I've been trying to split off some smaller changes from #108637 that might fix this, but haven't found anything yet, that change isn't really safe enough to backport.

I'm not following the logic here--the new hair system was a major feature in 3.5 and something that a lot of artists have been waiting for for years. The "Add rest position" feature is critical for an animated hair/fur workflow and was included in 3.5, then broken in 3.6. It seems strange to me that we would have an LTS release that breaks an existing feature.

As far as crashes go, I get lots and lots of crashes in general and I chalk them up to Blender being blender when I have lots going on in a scene, but now I'm going to look more closely and see if #109591 has been the root of my most recent issues.

> Actually, I suggest we just revert that commit in 82ca3d3604 or consider this a known issue in 3.6. "Add rest position" is quite a new feature that is only really helpful for the new hair system anyway. I've been trying to split off some smaller changes from #108637 that might fix this, but haven't found anything yet, that change isn't really safe enough to backport. I'm not following the logic here--the new hair system was a major feature in 3.5 and something that a lot of artists have been waiting for for years. The "Add rest position" feature is critical for an animated hair/fur workflow and was included in 3.5, then broken in 3.6. It seems strange to me that we would have an LTS release that breaks an existing feature. As far as crashes go, I get lots and lots of crashes in general and I chalk them up to Blender being blender when I have lots going on in a scene, but now I'm going to look more closely and see if #109591 has been the root of my most recent issues.

Well, that's definitely one of the crashes I've been hitting lately. I feel frustrated that something that was working fine in 3.5 is broken in 3.6 LTS, which is supposed to be more stable. Users who want a stable, animation-ready hair system will have to wait until the next LTS release. Will the fix be backported from 4.0? I don't know what a "critical fix" is as described on https://www.blender.org/download/lts/

Well, that's definitely one of the crashes I've been hitting lately. I feel frustrated that something that was working fine in 3.5 is broken in 3.6 LTS, which is supposed to be more stable. Users who want a stable, animation-ready hair system will have to wait until the next LTS release. Will the fix be backported from 4.0? I don't know what a "critical fix" is as described on https://www.blender.org/download/lts/
Member

I feel frustrated that something that was working fine in 3.5 is broken in 3.6 LTS, which is supposed to be more stable. Users who want a stable, animation-ready hair system will have to wait until the next LTS release. Will the fix be backported from 4.0? I don't know what a "critical fix" is as described on https://www.blender.org/download/lts/

It seems strange to me that we would have an LTS release that breaks an existing feature.

Just to clarify on 3.6 being LTS: the thing broke between 3.5 and 3.6. We dont guarantee 3.6 LTS being more stable than 3.5, what we should guarantee is that 3.6.X is more stable than 3.6.0.

That being said I certainly hope something will get backported, but for that this task is still open with High Priority. And because of this being High Priority, I would urge @HooglyBoogly , @JacquesLucke ( and maybe @brecht ) to make a decision soon if we

  • revert 82ca3d3604 (meaning rest position would be broken in 3.6LTS)
  • accept this report/bug here in 3.6LTS
  • come up with a fix for 3.6

The first two really seem suboptimal to me.

> I feel frustrated that something that was working fine in 3.5 is broken in 3.6 LTS, which is supposed to be more stable. Users who want a stable, animation-ready hair system will have to wait until the next LTS release. Will the fix be backported from 4.0? I don't know what a "critical fix" is as described on https://www.blender.org/download/lts/ >It seems strange to me that we would have an LTS release that breaks an existing feature. Just to clarify on 3.6 being LTS: the thing broke between 3.5 and 3.6. We dont guarantee 3.6 LTS being more stable than 3.5, what we should guarantee is that 3.6.X is more stable than 3.6.0. That being said I certainly hope something will get backported, but for that this task is still open with High Priority. And because of this being High Priority, I would urge @HooglyBoogly , @JacquesLucke ( and maybe @brecht ) to make a decision soon if we - revert 82ca3d3604 (meaning rest position would be broken in 3.6LTS) - accept this report/bug here in 3.6LTS - come up with a fix for 3.6 The first two really seem suboptimal to me.
Hans Goudey added
Type
Bug
and removed
Type
Report
labels 2023-08-23 15:43:42 +02:00
Blender Bot added
Status
Archived
and removed
Status
Confirmed
labels 2023-08-23 15:43:53 +02:00
Hans Goudey added
Status
Resolved
and removed
Status
Archived
labels 2023-08-23 15:44:01 +02:00
Sign in to join this conversation.
No Label
Interest
Alembic
Interest
Animation & Rigging
Interest
Asset System
Interest
Audio
Interest
Automated Testing
Interest
Blender Asset Bundle
Interest
BlendFile
Interest
Code Documentation
Interest
Collada
Interest
Compatibility
Interest
Compositing
Interest
Core
Interest
Cycles
Interest
Dependency Graph
Interest
Development Management
Interest
EEVEE
Interest
Freestyle
Interest
Geometry Nodes
Interest
Grease Pencil
Interest
ID Management
Interest
Images & Movies
Interest
Import Export
Interest
Line Art
Interest
Masking
Interest
Metal
Interest
Modeling
Interest
Modifiers
Interest
Motion Tracking
Interest
Nodes & Physics
Interest
OpenGL
Interest
Overlay
Interest
Overrides
Interest
Performance
Interest
Physics
Interest
Pipeline, Assets & IO
Interest
Platforms, Builds & Tests
Interest
Python API
Interest
Render & Cycles
Interest
Render Pipeline
Interest
Sculpt, Paint & Texture
Interest
Text Editor
Interest
Translations
Interest
Triaging
Interest
Undo
Interest
USD
Interest
User Interface
Interest
UV Editing
Interest
VFX & Video
Interest
Video Sequencer
Interest
Viewport & EEVEE
Interest
Virtual Reality
Interest
Vulkan
Interest
Wayland
Interest
Workbench
Interest: X11
Legacy
Asset Browser Project
Legacy
Blender 2.8 Project
Legacy
Milestone 1: Basic, Local Asset Browser
Legacy
OpenGL Error
Meta
Good First Issue
Meta
Papercut
Meta
Retrospective
Meta
Security
Module
Animation & Rigging
Module
Core
Module
Development Management
Module
Grease Pencil
Module
Modeling
Module
Nodes & Physics
Module
Pipeline, Assets & IO
Module
Platforms, Builds & Tests
Module
Python API
Module
Render & Cycles
Module
Sculpt, Paint & Texture
Module
Triaging
Module
User Interface
Module
VFX & Video
Module
Viewport & EEVEE
Platform
FreeBSD
Platform
Linux
Platform
macOS
Platform
Windows
Severity
High
Severity
Low
Severity
Normal
Severity
Unbreak Now!
Status
Archived
Status
Confirmed
Status
Duplicate
Status
Needs Info from Developers
Status
Needs Information from User
Status
Needs Triage
Status
Resolved
Type
Bug
Type
Design
Type
Known Issue
Type
Patch
Type
Report
Type
To Do
No Milestone
No project
No Assignees
7 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#109432
No description provided.