From 7ed47bec3b016cbd608aaea64687a1b919615670 Mon Sep 17 00:00:00 2001
From: Daniel Friesel <derf@finalrewind.org>
Date: Sun, 4 Jul 2021 13:19:50 +0200
Subject: map: decrease stop circle size when stops are close together

---
 lib/DBInfoscreen/Controller/Map.pm | 34 +++++++++++++++++++++++++---------
 templates/route_map.html.ep        |  2 +-
 2 files changed, 26 insertions(+), 10 deletions(-)

diff --git a/lib/DBInfoscreen/Controller/Map.pm b/lib/DBInfoscreen/Controller/Map.pm
index 8912e5d..cea5155 100644
--- a/lib/DBInfoscreen/Controller/Map.pm
+++ b/lib/DBInfoscreen/Controller/Map.pm
@@ -224,7 +224,7 @@ sub estimate_train_positions {
 
 # Input:
 #   now: DateTime
-#   route: hash
+#   route: arrayref of hashrefs
 #     lat: float
 #     lon: float
 #     name: str
@@ -241,9 +241,13 @@ sub estimate_train_positions2 {
 
 	my @train_positions;
 	my $next_stop;
+	my $geo                    = Geo::Distance->new;
+	my $stop_distance_sum      = 0;
+	my $avg_inter_stop_beeline = 0;
 
 	for my $i ( 1 .. $#route ) {
-		if (    ( $route[$i]{arr} // $route[$i]{dep} )
+		if (    not $next_stop
+			and ( $route[$i]{arr} // $route[$i]{dep} )
 			and ( $route[ $i - 1 ]{dep} // $route[ $i - 1 ]{arr} )
 			and $now > ( $route[ $i - 1 ]{dep} // $route[ $i - 1 ]{arr} )
 			and $now < ( $route[$i]{arr} // $route[$i]{dep} ) )
@@ -264,9 +268,9 @@ sub estimate_train_positions2 {
 				type    => 'next',
 				station => $route[$i],
 			};
-			last;
 		}
-		if ( ( $route[ $i - 1 ]{dep} // $route[ $i - 1 ]{arr} )
+		if (    not $next_stop
+			and ( $route[ $i - 1 ]{dep} // $route[ $i - 1 ]{arr} )
 			and $now <= ( $route[ $i - 1 ]{dep} // $route[ $i - 1 ]{arr} ) )
 		{
 			@train_positions
@@ -275,8 +279,17 @@ sub estimate_train_positions2 {
 				type    => 'present',
 				station => $route[ $i - 1 ],
 			};
-			last;
 		}
+		$stop_distance_sum += $geo->distance(
+			'meter',
+			$route[ $i - 1 ]{lon},
+			$route[ $i - 1 ]{lat},
+			$route[$i]{lon}, $route[$i]{lat}
+		);
+	}
+
+	if ($#route) {
+		$avg_inter_stop_beeline = $stop_distance_sum / $#route;
 	}
 
 	if ( @route and not $next_stop ) {
@@ -290,9 +303,10 @@ sub estimate_train_positions2 {
 	my $position_now = shift @train_positions;
 
 	return {
-		next_stop    => $next_stop,
-		position_now => $position_now,
-		positions    => \@train_positions,
+		next_stop              => $next_stop,
+		avg_inter_stop_beeline => $avg_inter_stop_beeline,
+		position_now           => $position_now,
+		positions              => \@train_positions,
 	};
 }
 
@@ -809,7 +823,9 @@ sub route {
 					}
 				],
 				station_coordinates => [@station_coordinates],
-				markers             => [@markers],
+				station_radius =>
+				  ( $train_pos->{avg_inter_stop_beeline} > 500 ? 250 : 100 ),
+				markers => [@markers],
 			);
 		}
 	)->catch(
diff --git a/templates/route_map.html.ep b/templates/route_map.html.ep
index 5bc7fd6..2f35b54 100644
--- a/templates/route_map.html.ep
+++ b/templates/route_map.html.ep
@@ -44,7 +44,7 @@ for (var station_id in stations) {
 		opacity: 0.7,
 		fillColor: '#f03',
 		fillOpacity: 0.5,
-		radius: 250
+		radius: <%= stash('station_radius') || 250 %>
 	}).bindPopup(stations[station_id][1].join('<br/>')).addTo(map);
 }
 
-- 
cgit v1.2.3