Merge pull request #1863 from hannobraun/model

Clean up model-related code in `fj-interop`

crates/fj-interop/src/lib.rs

@@ -11,7 +11,6 @@
 
 #![warn(missing_docs)]
 
-pub mod debug;
 pub mod ext;
 pub mod mesh;
-pub mod processed_shape;
+pub mod model;

crates/fj-interop/src/model.rs

@@ -0,0 +1,15 @@
+//! An approximated model
+
+use fj_math::{Aabb, Point};
+
+use crate::mesh::Mesh;
+
+/// An approximated model
+#[derive(Clone, Debug)]
+pub struct Model {
+    /// The triangle mesh that approximates the model
+    pub mesh: Mesh<Point<3>>,
+
+    /// The axis-aligned bounding box of the model
+    pub aabb: Aabb<3>,
+}

crates/fj-viewer/src/camera.rs

@@ -1,7 +1,7 @@
 //! Viewer camera module
 use std::f64::consts::FRAC_PI_2;
 
-use fj_interop::{mesh::Mesh, processed_shape::ProcessedShape};
+use fj_interop::{mesh::Mesh, model::Model};
 use fj_math::{Aabb, Point, Scalar, Transform, Vector};
 
 use crate::screen::NormalizedScreenPosition;
@@ -82,7 +82,7 @@ impl Camera {
     pub fn focus_point(
         &self,
         cursor: Option<NormalizedScreenPosition>,
-        shape: &ProcessedShape,
+        shape: &Model,
     ) -> FocusPoint {
         self.calculate_focus_point(cursor, &shape.mesh)
             .unwrap_or_else(|| FocusPoint(shape.aabb.center()))

crates/fj-viewer/src/viewer.rs

@@ -1,4 +1,4 @@
-use fj_interop::processed_shape::ProcessedShape;
+use fj_interop::model::Model;
 use fj_math::Aabb;
 use tracing::warn;
 
@@ -28,7 +28,7 @@ pub struct Viewer {
     pub renderer: Renderer,
 
     /// The shape
-    pub shape: Option<ProcessedShape>,
+    pub shape: Option<Model>,
 }
 
 impl Viewer {
@@ -60,7 +60,7 @@ impl Viewer {
     }
 
     /// Handle the shape being updated
-    pub fn handle_shape_update(&mut self, shape: ProcessedShape) {
+    pub fn handle_shape_update(&mut self, shape: Model) {
         self.renderer.update_geometry((&shape.mesh).into());
 
         let aabb = shape.aabb;

crates/fj-window/src/display.rs

@@ -1,6 +1,4 @@
-use fj_interop::{
-    debug::DebugInfo, mesh::Mesh, processed_shape::ProcessedShape,
-};
+use fj_interop::{mesh::Mesh, model::Model};
 use fj_math::{Aabb, Point};
 use fj_viewer::{
     InputEvent, NormalizedScreenPosition, RendererInitError, Screen,
@@ -24,10 +22,9 @@ pub fn display(mesh: Mesh<Point<3>>, invert_zoom: bool) -> Result<(), Error> {
     let window = Window::new(&event_loop)?;
     let mut viewer = block_on(Viewer::new(&window))?;
 
-    viewer.handle_shape_update(ProcessedShape {
+    viewer.handle_shape_update(Model {
         aabb: Aabb::<3>::from_points(mesh.vertices()),
         mesh,
-        debug_info: DebugInfo::new(),
     });
 
     let mut held_mouse_button = None;