Fix asterisk shape

Author: thomasp85

CHANGELOG.md

@@ -15,8 +15,9 @@ skill for llms through the `skill` command (#361)
 - Rendering of inline plots in Positron had a bad interaction with how we
 handled auto-resizing in the plot pane. We now have a per-output-location path
 in the Jupyter kernel (#360)
-- Passing the shape aesthetic via `SETTING` now correctly translates named 
+- Passing the shape aesthetic via `SETTING` now correctly translates named
 shapes (#368)
+- Asterisk shape now has lines 60 degrees apart, giving an even shape
 
 ### Changed
 

src/plot/scale/shape.rs

@@ -162,13 +162,16 @@ fn vline_coords() -> Vec<Vec<(f64, f64)>> {
     vec![vec![(0.0, -0.8), (0.0, 0.8)]]
 }
 
-/// Asterisk (*) - three lines through center.
+/// Asterisk (*) - three lines through center evenly spaced at 60° (six wedges).
 fn asterisk_coords() -> Vec<Vec<(f64, f64)>> {
-    vec![
-        vec![(-0.8, 0.0), (0.8, 0.0)],  // horizontal
-        vec![(-0.6, -0.7), (0.6, 0.7)], // diagonal /
-        vec![(-0.6, 0.7), (0.6, -0.7)], // diagonal \
-    ]
+    let r: f64 = 0.8;
+    (0..3)
+        .map(|i| {
+            let angle = (i as f64) * std::f64::consts::PI / 3.0;
+            let (sin, cos) = angle.sin_cos();
+            vec![(-r * cos, -r * sin), (r * cos, r * sin)]
+        })
+        .collect()
 }
 
 /// Bowtie - two triangles meeting at center.