Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement: Internal Bridge angle override #7908

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

igiannakas
Copy link
Contributor

@igiannakas igiannakas commented Jan 3, 2025

Description

Enable overriding of the auto calculated internal bridge angle for cases where the automated algorithm fails to produce adequate bridging results.

Set to 0 (default) to disable (retain the automated calculation).

Fixes #2186

Screenshots/Recordings/Graphs

Default
image

Override:
image

Tests

  1. Tested with 0 value - works as expected. Automatically calculated bridge angle used
  2. Tested with any > 0 set value. Works as expected
  3. Used modifier on the object to set override. Works, but only if the entire bridging layer is covered by the modifier.

@Noisyfox would welcome your thoughts please on point 3. When placing a modifier it segments the region correctly but it ignores the angle provided. Unfortunately I am not very familiar with this (quite complex) code so will end up breaking something if I interfere more :) any pointers on where to look for this would be greatly appreciated :)

Copy link

@laukejas laukejas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and it seems to be working without any issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants