Scale to zero in object mode unable to select #77559

New Issue

iosif · 2020-06-07T07:33:10+02:00

iosif commented

2020-06-07 07:33:10 +02:00

System Information
Operating system: windows 8
Graphics card: Nvidia gforce gtx 570

Blender Version
Broken: 2.83 official release, 2.82a official release
Worked: 2.78 , 2.79a

Short description of error
If you scale an object to zero the object can't be selected unless you click on origin - if you are in wireframe shading, (works in solid shading).

Exact steps for others to reproduce the error
Based on the default startup or an attached .blend file (as simple as possible).
Default cube selected in object mode:
S Key - to scale
Z Key - constrain to z axis
0 Key - to flat it out
AA - to deselect it
Make sure you are in wireframe shading
You can't selected unless you click on origin or you change to solid shading!

If you go in edit mode before you scale it to zero it works.
If you scale it to zero in object mode, selected by origin or in solid shading or in outliner and apply scale it became selectable as expected.

**System Information** Operating system: windows 8 Graphics card: Nvidia gforce gtx 570 **Blender Version** Broken: 2.83 official release, 2.82a official release Worked: 2.78 , 2.79a **Short description of error** If you scale an object to zero the object can't be selected unless you click on origin - if you are in wireframe shading, (works in solid shading). **Exact steps for others to reproduce the error** Based on the default startup or an attached .blend file (as simple as possible). Default cube selected in object mode: S Key - to scale Z Key - constrain to z axis 0 Key - to flat it out AA - to deselect it Make sure you are in wireframe shading You can't selected unless you click on origin or you change to solid shading! If you go in edit mode before you scale it to zero it works. If you scale it to zero in object mode, selected by origin or in solid shading or in outliner and **apply scale** it became selectable as expected.

iosif commented

2020-06-07 07:33:10 +02:00

Added subscriber: @Sisel

Deep Majumder commented

2020-06-07 08:32:51 +02:00

Added subscriber: @dmcoder

Deep Majumder commented

2020-06-07 08:32:51 +02:00

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

Deep Majumder commented

2020-06-07 08:32:51 +02:00

I can confirm that the bug still persists in the master build (a39cc5ae4d).

I can confirm that the bug still persists in the master build (a39cc5ae4d).

Jacques Lucke commented

2020-06-08 17:29:08 +02:00

Added subscriber: @JacquesLucke

Jacques Lucke commented

2020-06-08 17:29:08 +02:00

I can confirm it as well.

Clément Foucault commented

2020-07-02 15:41:30 +02:00

Added subscribers: @brecht, @fclem

Clément Foucault commented

2020-07-02 15:41:30 +02:00

On linux + AMD it only happens on Mesa (opensource driver) but not AMD pro driver.

Also the cube turns black in this case.

This is because the matrix is not invertible in this case and the inverse matrix is just set to zeros.

@brecht not sure what can we do in this case.

On linux + AMD it only happens on Mesa (opensource driver) but not AMD pro driver. Also the cube turns black in this case. This is because the matrix is not invertible in this case and the inverse matrix is just set to zeros. @brecht not sure what can we do in this case.

Brecht Van Lommel commented

2020-07-02 16:19:26 +02:00

What we could do is first try to invert as normal, and if that fails use the pseudoinverse instead. We do this with pseudoinverse_m3_m3 in the transform code.

What we could do is first try to invert as normal, and if that fails use the pseudoinverse instead. We do this with `pseudoinverse_m3_m3` in the transform code.

Clément Foucault commented

2020-07-02 16:29:45 +02:00

This fixes the issue but i'm not sure about the performance impact.

@@ -143,11 +143,11 @@ void BKE_object_eval_constraints(Depsgraph *depsgraph, Scene *scene, Object *ob)
 void BKE_object_eval_transform_final(Depsgraph *depsgraph, Object *ob)
 {
   DEG_debug_print_eval(depsgraph, __func__, ob->id.name, ob);
   /* Make sure inverse matrix is always up to date. This way users of it
    * do not need to worry about recalculating it. */
-  invert_m4_m4(ob->imat, ob->obmat);
+  invert_m4_m4_safe(ob->imat, ob->obmat);
   /* Set negative scale flag in object. */
   if (is_negative_m4(ob->obmat)) {
     ob->transflag |= OB_NEG_SCALE;
   }
   else {

This fixes the issue but i'm not sure about the performance impact. ``` @@ -143,11 +143,11 @@ void BKE_object_eval_constraints(Depsgraph *depsgraph, Scene *scene, Object *ob) void BKE_object_eval_transform_final(Depsgraph *depsgraph, Object *ob) { DEG_debug_print_eval(depsgraph, __func__, ob->id.name, ob); /* Make sure inverse matrix is always up to date. This way users of it * do not need to worry about recalculating it. */ - invert_m4_m4(ob->imat, ob->obmat); + invert_m4_m4_safe(ob->imat, ob->obmat); /* Set negative scale flag in object. */ if (is_negative_m4(ob->obmat)) { ob->transflag |= OB_NEG_SCALE; } else { ```

Brecht Van Lommel commented

2020-07-02 18:47:52 +02:00

Added subscribers: @Sergey, @ideasman42

Brecht Van Lommel commented

2020-07-02 18:47:52 +02:00

I think it would be a good thing to invert the object matrix like this. There are many places that do this though, and they would all need to be updated (see e.g. grep -r "invert_m4_m4.*obmat" source/blender/).

@Sergey, @ideasman42, any opinion on doing this?

Could we perhaps even change invert_m4_m4 itself to always behave like invert_m4_m4_safe?

I think it would be a good thing to invert the object matrix like this. There are many places that do this though, and they would all need to be updated (see e.g. `grep -r "invert_m4_m4.*obmat" source/blender/`). @Sergey, @ideasman42, any opinion on doing this? Could we perhaps even change `invert_m4_m4` itself to always behave like `invert_m4_m4_safe`?

blender-admin commented

2020-07-03 00:20:21 +02:00

This issue was referenced by 5a13f682ee

This issue was referenced by 5a13f682ee5f55717c2fbc35371cd5bf58829b88

Clément Foucault commented

2020-07-03 00:21:39 +02:00

I'll commit this fix. A more global fix could be committed separately.

Clément Foucault commented

2020-07-03 00:22:15 +02:00

Changed status from 'Confirmed' to: 'Resolved'

Clément Foucault closed this issue

2020-07-03 00:22:15 +02:00

Clément Foucault self-assigned this 2020-07-03 00:22:15 +02: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

Scale to zero in object mode unable to select #77559