From 2cc4c52a596557bb99e8211d890d7cc25d4eff79 Mon Sep 17 00:00:00 2001
From: Mikolaj Wielgus <wielgusmikolaj@gmail.com>
Date: Mon, 8 Jun 2026 00:43:06 +0200
Subject: [PATCH] Use better attraction cost function

---
 topola-egui/src/menu_bar.rs  |  4 ++--
 topola/src/autoplacer/mod.rs | 14 ++++++++------
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/topola-egui/src/menu_bar.rs b/topola-egui/src/menu_bar.rs
index 14a7e9a..b2ea1c4 100644
--- a/topola-egui/src/menu_bar.rs
+++ b/topola-egui/src/menu_bar.rs
@@ -138,10 +138,10 @@ impl MenuBar {
                         controller.workspace.board_mut(),
                         AutoplacerSchedule {
                             initial_temperature: 1000.0,
-                            temperature_common_ratio: 0.95,
+                            temperature_common_ratio: 0.98,
                             initial_std_dev: 1000.0,
                             std_dev_common_ratio: 0.995,
-                            max_steps: 200,
+                            max_steps: 500,
                         },
                     );
                 }
diff --git a/topola/src/autoplacer/mod.rs b/topola/src/autoplacer/mod.rs
index f8c55fc..32b355d 100644
--- a/topola/src/autoplacer/mod.rs
+++ b/topola/src/autoplacer/mod.rs
@@ -11,7 +11,7 @@ use rand_distr::{Distribution, Normal};
 use undoredo::{FlushDelta, ResetDelta};
 
 use crate::{
-    board::{Board, BoardDelta},
+    board::Board,
     layout::{Layout, compounds::ComponentId},
     orientation::Orientation,
     selections::ComponentSelection,
@@ -36,7 +36,6 @@ pub struct Autoplacer {
     components: Vec<ComponentId>,
     schedule: AutoplacerSchedule,
     step_counter: u64,
-    origin_delta: BoardDelta, //rng: ThreadRng,
 }
 
 impl Autoplacer {
@@ -49,7 +48,6 @@ impl Autoplacer {
             components: board.resolve_components(selection).collect(),
             schedule,
             step_counter: 0,
-            origin_delta: board.flush_delta(),
         }
     }
 
@@ -171,7 +169,7 @@ impl Autoplacer {
         crate::profile_function!();
         layout
             .locate_component_repulsions(component, Orientation::Oblique)
-            .map(|vector| vector.x.abs() + vector.y.abs())
+            .map(|vector| 1000 * (vector.x.abs() + vector.y.abs()))
             .sum()
     }
 
@@ -179,7 +177,11 @@ impl Autoplacer {
         crate::profile_function!();
         layout
             .component_attractions(component)
-            .map(|vector| 1.0 / (1.0 + (vector.x.abs() + vector.y.abs()) as f64))
+            .map(|vector| {
+                (vector.x.abs().pow(2) as f64 + vector.y.abs().pow(2) as f64)
+                    .sqrt()
+                    .powf(0.5)
+            })
             .sum()
     }
 
@@ -187,7 +189,7 @@ impl Autoplacer {
         crate::profile_function!();
         layout
             .component_retentions(component)
-            .map(|vector| 100 * (vector.x.abs() + vector.y.abs()))
+            .map(|vector| 1000 * (vector.x.abs() + vector.y.abs()))
             .sum()
     }