From ddb47e1ad45bb7751ac63c55be9a4c8cac019ace Mon Sep 17 00:00:00 2001
From: Chris Tsang <chris.2y3@outlook.com>
Date: Fri, 29 Mar 2024 18:58:06 +0000
Subject: [PATCH] Revert "Experiment with idealizing small circles"

This reverts commit c3012c6aefd2d06effcd7bdeeaa2b5500a63ab47.
---
 cmdapp/Cargo.toml       |  2 +-
 cmdapp/src/converter.rs | 37 ++++++++++---------------------------
 2 files changed, 11 insertions(+), 28 deletions(-)

diff --git a/cmdapp/Cargo.toml b/cmdapp/Cargo.toml
index 7a50aa6..6def91b 100644
--- a/cmdapp/Cargo.toml
+++ b/cmdapp/Cargo.toml
@@ -13,7 +13,7 @@ keywords = ["svg", "computer-graphics"]
 [dependencies]
 clap = "2.33.3"
 image = "0.23.10"
-visioncortex = { version = "0.8.4" }
+visioncortex = { version = "0.8.1" }
 fastrand = "1.8"
 pyo3 = { version = "0.19.0", optional = true }
 
diff --git a/cmdapp/src/converter.rs b/cmdapp/src/converter.rs
index c31a0c0..d0c4c2b 100644
--- a/cmdapp/src/converter.rs
+++ b/cmdapp/src/converter.rs
@@ -5,17 +5,13 @@ use super::config::{ColorMode, Config, ConverterConfig, Hierarchical};
 use super::svg::SvgFile;
 use fastrand::Rng;
 use visioncortex::color_clusters::{KeyingAction, Runner, RunnerConfig, HIERARCHICAL_MAX};
-use visioncortex::{
-    approximate_circle_with_spline, Color, ColorImage, ColorName, CompoundPath, PathSimplifyMode,
-};
+use visioncortex::{Color, ColorImage, ColorName};
 
 const NUM_UNUSED_COLOR_ITERATIONS: usize = 6;
 /// The fraction of pixels in the top/bottom rows of the image that need to be transparent before
 /// the entire image will be keyed.
 const KEYING_THRESHOLD: f32 = 0.2;
 
-const SMALL_CIRCLE: i32 = 12;
-
 /// Convert an in-memory image into an in-memory SVG
 pub fn convert(img: ColorImage, config: Config) -> Result<SvgFile, String> {
     let config = config.into_converter_config();
@@ -171,28 +167,15 @@ fn color_image_to_svg(mut img: ColorImage, config: ConverterConfig) -> Result<Sv
     let mut svg = SvgFile::new(width, height, config.path_precision);
     for &cluster_index in view.clusters_output.iter().rev() {
         let cluster = view.get_cluster(cluster_index);
-        let paths = if matches!(config.mode, PathSimplifyMode::Spline)
-            && cluster.rect.width() < SMALL_CIRCLE
-            && cluster.rect.height() < SMALL_CIRCLE
-            && cluster.to_shape(&view).is_circle()
-        {
-            let mut paths = CompoundPath::new();
-            paths.add_spline(approximate_circle_with_spline(
-                cluster.rect.left_top(),
-                cluster.rect.width(),
-            ));
-            paths
-        } else {
-            cluster.to_compound_path(
-                &view,
-                false,
-                config.mode,
-                config.corner_threshold,
-                config.length_threshold,
-                config.max_iterations,
-                config.splice_threshold,
-            )
-        };
+        let paths = cluster.to_compound_path(
+            &view,
+            false,
+            config.mode,
+            config.corner_threshold,
+            config.length_threshold,
+            config.max_iterations,
+            config.splice_threshold,
+        );
         svg.add_path(paths, cluster.residue_color());
     }