Adding archimesh stairs object to the scene flips normals of all other objects in scene #67048

Closed
opened 2019-07-16 12:25:52 +02:00 by Ludvik Koutny · 11 comments

System Information
Operating system: Windows-10-10.0.17763 64 Bits
Graphics card: GeForce GTX 1080 Ti/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 430.64

Blender Version
Broken: version: 2.80 (sub 74), branch: master, commit date: 2019-07-11 13:50, hash: blender/blender@06312c6d2d
Worked: (optional)

Short description of error
When adding a stairs object from archimesh (addon bundled with Blender), once stairs object is added, it flips normals of all the other objects in scene.

Exact steps for others to reproduce the error

  1. Enable archimesh addon
  2. Enable backface culling in viewport setting to see the normal orientation
  3. Add archimesh stairs to the scene

Result: Normals of all the scene objects are flipped
Expected: Normals of other scene objects remain unaffected

**System Information** Operating system: Windows-10-10.0.17763 64 Bits Graphics card: GeForce GTX 1080 Ti/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 430.64 **Blender Version** Broken: version: 2.80 (sub 74), branch: master, commit date: 2019-07-11 13:50, hash: `blender/blender@06312c6d2d` Worked: (optional) **Short description of error** When adding a stairs object from archimesh (addon bundled with Blender), once stairs object is added, it flips normals of all the other objects in scene. **Exact steps for others to reproduce the error** 1. Enable archimesh addon 2. Enable backface culling in viewport setting to see the normal orientation 3. Add archimesh stairs to the scene Result: Normals of all the scene objects are flipped Expected: Normals of other scene objects remain unaffected
Author

Added subscriber: @Rawalanche

Added subscriber: @Rawalanche
Member

Added subscriber: @lichtwerk

Added subscriber: @lichtwerk
Member

Cannot reproduce here.
[Stairs themselves have flipped normals on the side :), but thats another story... existing objects remain untouched afaict...]
[also dont see any recent commits that might have changed this...]
#67048.png

Do you have a .blend for us where this happens?

Cannot reproduce here. [Stairs themselves have flipped normals on the side :), but thats another story... existing objects remain untouched afaict...] [also dont see any recent commits that might have changed this...] ![#67048.png](https://archive.blender.org/developer/F7612777/T67048.png) Do you have a .blend for us where this happens?
Author

In #67048#723751, @lichtwerk wrote:
Cannot reproduce here.
[Stairs themselves have flipped normals on the side :), but thats another story... existing objects remain untouched afaict...]
[also dont see any recent commits that might have changed this...]
#67048.png

Do you have a .blend for us where this happens?

Yes, but it's a job file. Can't share it publicly, but I can send you privately. Where to?

> In #67048#723751, @lichtwerk wrote: > Cannot reproduce here. > [Stairs themselves have flipped normals on the side :), but thats another story... existing objects remain untouched afaict...] > [also dont see any recent commits that might have changed this...] > ![#67048.png](https://archive.blender.org/developer/F7612777/T67048.png) > > Do you have a .blend for us where this happens? Yes, but it's a job file. Can't share it publicly, but I can send you privately. Where to?
Member

This is not reproducable with a sharable file?

otherwise: are you on https://blender.chat/channel/blender-coders? (think we can send files one-to-one there...)

This is not reproducable with a sharable file? otherwise: are you on https://blender.chat/channel/blender-coders? (think we can send files one-to-one there...)
Member

also, does this still happen with Factory Settings and just Archimesh enabled?

also, does this still happen with Factory Settings and **just Archimesh** enabled?
Author

In #67048#723774, @lichtwerk wrote:
This is not reproducable with a sharable file?

otherwise: are you on https://blender.chat/channel/blender-coders? (think we can send files one-to-one there...)

I am unable to access that site. It just keeps loading infinitely.

Try following:

  1. Enable archimesh addon
  2. In an empty, new .blend file select default cube, go to edit mode, and recalculate normals inside (Mesh>Normals>Recalculate Inside)
  3. Exit edit mode back to object mode, and enable backface culling in Viewport Shading settings. Now you should see interior faces of the cube, as normals are inverted
  4. Add Archimesh Stairs object, and you should observe the face rendering immediately flip, so that you see outside faces despite normals being inside.
> In #67048#723774, @lichtwerk wrote: > This is not reproducable with a sharable file? > > otherwise: are you on https://blender.chat/channel/blender-coders? (think we can send files one-to-one there...) I am unable to access that site. It just keeps loading infinitely. Try following: 1. Enable archimesh addon 2. In an empty, new .blend file select default cube, go to edit mode, and recalculate normals inside (Mesh>Normals>Recalculate Inside) 3. Exit edit mode back to object mode, and enable backface culling in Viewport Shading settings. Now you should see interior faces of the cube, as normals are inverted 4. Add Archimesh Stairs object, and you should observe the face rendering immediately flip, so that you see outside faces despite normals being inside.
Member

OKi, can repro now...

OKi, can repro now...
Philipp Oeser self-assigned this 2019-07-16 17:04:37 +02:00
Member

Bad usages of bpy.ops.object.select_all() in archimesh it seems, I wonder why it is not complaining if called with False...
All is good if you do bpy.ops.object.select_all(action='DESELECT'), this might not be needed even (stuff is deselected prior anyways...).
Will check on all this in a bit...

Bad usages of `bpy.ops.object.select_all()` in archimesh it seems, I wonder why it is not complaining if called with `False`... All is good if you do `bpy.ops.object.select_all(action='DESELECT')`, this might not be needed even (stuff is deselected prior anyways...). Will check on all this in a bit...

This issue was referenced by 184f7341bd

This issue was referenced by 184f7341bd171d411a326de6b4804b61c9903ad2
Member

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 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-addons#67048
No description provided.