app-proto/src/assembly.rs

@@ -127,7 +127,7 @@ pub trait Element: Serial + ProblemPoser + DisplayItem {
 }
 
 impl Debug for dyn Element {
-    fn fmt(&self, f: &mut Formatter<'_>) -> fmt::Result {
+    fn fmt(&self, f: &mut Formatter<'_>) -> Result<(), fmt::Error> {
         self.id().fmt(f)
     }
 }
@@ -308,11 +308,10 @@ impl Element for Point {
 
     fn default_regulators(self: Rc<Self>) -> Vec<Rc<dyn Regulator>> {
         all::<Axis>()
-            .map(
-                |axis| Rc::new(
-                    PointCoordinateRegulator::new(self.clone(), axis)
-                ) as Rc::<dyn Regulator>
-            )
+            .map(|axis| {
+                Rc::new(PointCoordinateRegulator::new(self.clone(), axis))
+                    as Rc::<dyn Regulator>
+            })
             .collect()
     }
     
@@ -515,15 +514,7 @@ impl ProblemPoser for HalfCurvatureRegulator {
 pub enum Axis {X = 0, Y = 1, Z = 2}
 
 impl Axis {
-    fn name(&self) -> &'static str {
-        match self { Axis::X => "X", Axis::Y => "Y", Axis::Z => "Z" }
-    }
-}
-
-impl fmt::Display for Axis {
-    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
-        f.write_str(self.name())
-    }
+    pub const NAME: [&str; Axis::CARDINALITY] = ["X", "Y", "Z"];
 }
 
 pub struct PointCoordinateRegulator {
@@ -539,32 +530,19 @@ impl PointCoordinateRegulator {
         let measurement = subject.representation().map(
             move |rep| rep[axis as usize]
         );
-        
         let set_point = create_signal(SpecifiedValue::from_empty_spec());
-        let serial = Self::next_serial();
-        
-        Self { subject, axis, measurement, set_point, serial }
+        Self { subject, axis, measurement, set_point, serial: Self::next_serial() }
     }
 }
 
 impl Serial for PointCoordinateRegulator {
-    fn serial(&self) -> u64 {
-        self.serial
-    }
+    fn serial(&self) -> u64 { self.serial }
 }
 
 impl Regulator for PointCoordinateRegulator {
-    fn subjects(&self) -> Vec<Rc<dyn Element>> {
-        vec![self.subject.clone()]
-    }
-    
-    fn measurement(&self) -> ReadSignal<f64> {
-        self.measurement
-    }
-    
-    fn set_point(&self) -> Signal<SpecifiedValue> {
-        self.set_point
-    }
+    fn subjects(&self) -> Vec<Rc<dyn Element>> { vec![self.subject.clone()] }
+    fn measurement(&self) -> ReadSignal<f64> { self.measurement }
+    fn set_point(&self) -> Signal<SpecifiedValue> { self.set_point }
 }
 
 impl ProblemPoser for PointCoordinateRegulator {
@@ -572,25 +550,22 @@ impl ProblemPoser for PointCoordinateRegulator {
         self.set_point.with_untracked(|set_pt| {
             if let Some(val) = set_pt.value {
                 let col = self.subject.column_index().expect(
-                    "Subject should be indexed before point coordinate regulator writes problem data"
-                );
+                    "Subject must be indexed before point-coordinate regulator poses.");
                 problem.frozen.push(self.axis as usize, col, val);
-                
-                // if all three of the subject's spatial coordinates have been
-                // frozen, then freeze its norm component too
-                let mut coords_frozen = [0.0; Axis::CARDINALITY];
-                let mut n_set: usize = 0;
+                // Check if all three spatial coordinates have been frozen, and if so,
+                // freeze the norm component as well
+                let mut coords = [0.0; Axis::CARDINALITY];
+                let mut nset: usize = 0;
                 for &MatrixEntry {index, value} in &(problem.frozen) {
-                    let (row_frozen, col_frozen) = index;
-                    if col_frozen == col && row_frozen < Axis::CARDINALITY {
-                        n_set += 1;
-                        coords_frozen[row_frozen] = value
+                    if index.1 == col && index.0 < Axis::CARDINALITY {
+                        nset += 1;
+                        coords[index.0] = value
                     }
                 }
-                if n_set == Axis::CARDINALITY {
-                    let [x, y, z] = coords_frozen;
-                    let norm = point(x, y, z)[Point::NORM_COMPONENT];
-                    problem.frozen.push(Point::NORM_COMPONENT, col, norm);
+                if nset == Axis::CARDINALITY {
+                    let [x, y, z] = coords;
+                    problem.frozen.push(
+                        Point::NORM_COMPONENT, col, point(x,y,z)[Point::NORM_COMPONENT]);
                 }
             }
         });

app-proto/src/components/outline.rs

@@ -6,6 +6,7 @@ use web_sys::{KeyboardEvent, MouseEvent, wasm_bindgen::JsCast};
 use crate::{
     AppState,
     assembly::{
+        Axis,
         Element,
         HalfCurvatureRegulator,
         InversiveDistanceRegulator,
@@ -122,7 +123,7 @@ impl OutlineItem for HalfCurvatureRegulator {
 
 impl OutlineItem for PointCoordinateRegulator {
     fn outline_item(self: Rc<Self>, _element: &Rc<dyn Element>) -> View {
-        let name = format!("{} coordinate", self.axis);
+        let name = format!("{} coordinate", Axis::NAME[self.axis as usize]);
         view! {
             li(class = "regulator") {
                 div(class = "regulator-label") // for spacing