Skip to content

Sync ForceDirectedLayout and NodeEditor README tuning values with current defaults - #399

Merged
matt-edmondson merged 2 commits into
mainfrom
copilot/update-force-directed-layout-readme
Sep 13, 2026
Merged

matt-edmondson merged 2 commits into
mainfrom
copilot/update-force-directed-layout-readme

Conversation

Copilot AI commented Sep 13, 2026 •

Copy link
Copy Markdown
Contributor

The package/docs examples were still publishing pre-tuning physics values while nearby prose claimed the numbers were corpus-measured. This updates both README snippets to match LayoutSettings.Defaults and documents the internal floor applied to repulsion distance.

  • ForceDirectedLayout README: align published defaults

    • Updated the tuning block values to current defaults:
      • RepulsionStrength: 600_000.0 -> 900_000.0
      • LinkSpringStrength: 0.5 -> 0.1
      • RestLinkLength: 225.0 -> 50.0
      • DirectionalBias: 0.5 -> 4.0
      • LinkFlatteningStrength: 0.5 -> 3.0
      • MinRepulsionDistance: 50.0 -> 5.0
  • ForceDirectedLayout README: repulsion clamp behavior note

    • Added explicit note that MinRepulsionDistance is floored internally (MinimumRepulsionClamp) so inverse-square repulsion remains finite even if callers set zero.
  • ImGui.NodeEditor README: refresh stale illustrative physics snippet

    • Updated example values to the same tuned defaults direction:
      • RepulsionStrength: 1_200_000.0 -> 900_000.0
      • LinkSpringStrength: 0.5 -> 0.1
      • RestLinkLength: 225.0 -> 50.0
PhysicsSettings settings = new()
{
    Enabled = true,
    RepulsionStrength = 900_000.0,
    LinkSpringStrength = 0.1,
    RestLinkLength = 50.0,
    DirectionalBias = 4.0,
    LinkFlatteningStrength = 3.0,
    MinRepulsionDistance = 5.0,
};

Co-authored-by: matt-edmondson <19528727+matt-edmondson@users.noreply.github.com>
Copilot AI changed the title [WIP] Update ForceDirectedLayout README with correct tuning values Sync ForceDirectedLayout and NodeEditor README tuning values with current defaults Sep 13, 2026
@matt-edmondson
matt-edmondson marked this pull request as ready for review September 13, 2026 13:03
@matt-edmondson
matt-edmondson merged commit 2af924b into main Sep 13, 2026
6 checks passed
@matt-edmondson
matt-edmondson deleted the copilot/update-force-directed-layout-readme branch September 13, 2026 13:05
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.

ForceDirectedLayout/README.md publishes the pre-tuning defaults and then asserts they are the measured ones

2 participants