Skip to content

Styles revisited#7938

Open
falken10vdl wants to merge 10 commits intoIfcOpenShell:v0.8.0from
falken10vdl:styles-revisited
Open

Styles revisited#7938
falken10vdl wants to merge 10 commits intoIfcOpenShell:v0.8.0from
falken10vdl:styles-revisited

Conversation

@falken10vdl
Copy link
Copy Markdown
Contributor

@falken10vdl falken10vdl commented Apr 15, 2026

This is a proposal for a simple toggle to get either pretty styles based on external files (texture or .blend, UV, shaders.) or flat rendering based on Ifc simpler approach.

Kooha-2026-04-15-12-53-30.mp4

Cheers!

falken10vdl and others added 7 commits April 9, 2026 19:27
Opens a props dialog when running SuggestShadeFromExternalStyle,
exposing a Value offset (-1 to +1) and Saturation factor (0 to 2)
that are applied in HSV space before writing the IFC surface colour.

Generated with the assistance of an AI coding tool.
…xternal styles as well and set active based on toggle. We only travers teh files once and apply to all materials that share the style. Now we can have pretty mode as default :)
falken10vdl and others added 2 commits April 20, 2026 08:16
Switching to External style had no visible effect because
switch_shading used a fast-path that only flipped
is_active_output on the dual-branch nodes. This was a
no-op when BIM_Output_External was already active, and
material.update_tag() alone was insufficient to trigger
a viewport redraw regardless. Fix by always calling
activate_external_style for the External case, which
re-loads the .blend material and goes through Blender's
operator path that reliably refreshes the viewport.

Generated with the assistance of an AI coding tool.
AI effort: 4/10
@theoryshaw
Copy link
Copy Markdown
Member

I figured i'd just push this fix to your branch, although it's bug on v0.8.0 :)
falken10vdl#5

…al-viewport

Fix IfcOpenShell#7990 - Fix update_current_style not refreshing viewport
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants