Tidy up

[alexespencer]

No description provided.

[Copilot]

Pull Request Overview

This PR tidies up the visualize module by removing unused or redundant code and improving encapsulation in the Model structure. Key changes include removing the unused Technique enum, making some Model fields private and adding a setter for mouse_position, and updating related call sites throughout the codebase.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
visualize/src/model.rs	Removed Technique enum; encapsulated Model fields; updated draw_app logic.
visualize/src/main.rs	Replaced direct field assignments with setter calls; updated draw_app usage.
visualize/src/lib.rs	Removed outdated export of Technique to match changes in model.rs.

Comments suppressed due to low confidence (3)

visualize/src/model.rs:20

• Removal of the public Technique enum may break external consumers if they relied on it. Confirm that this export is no longer needed and update any dependent documentation or tests accordingly.

-pub enum Technique {

visualize/src/model.rs:23

• Changing 'points', 'mouse_position', and 'region' from public to private improves encapsulation; ensure that all required external interactions with these fields now use provided accessor or setter methods.

pub struct Model {

visualize/src/model.rs:70

• [nitpick] The update to draw_app now exclusively uses internal points, removing flexibility of drawing a custom set of points; verify that this change aligns with the overall design intentions.

pub fn draw_app(&self, draw: &Draw) {