v0.5.5

Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "satpaper"
 authors = ["Colonial"]
-version = "0.5.4"
+version = "0.5.5"
 edition = "2021"
 
 description = "Display near-real-time satellite imagery on your desktop."

README.md

@@ -112,7 +112,7 @@ Thanks to `cyberbit`, everything you need to build and run a Satpaper Docker ima
 - `-b`/`--background-image`/`SATPAPER_BACKGROUND_IMAGE` - the path to an image to use as the background.
     - Most common image formats are supported.
     - For best results, the image should match the specified resolution, but Satpaper will resize the image to fit if need be.
-    - Satpaper uses a basic "marching" algorithm to find the bounds of the Earth and apply transparency to the original image, but it's not perfect - some black bordering may remain.
+    - Satpaper uses a basic "marching" algorithm to find the bounds of the Earth and apply transparency to the original image, but it's not perfect - some black bordering and/or jagged edges may remain. (Unfortunately, the canonical algorithm for this problem - flood filling - doesn't really work, because it tends to end up eating into the Earth at night. If you have an idea for a better solution, please let me know!)
 - `-w`/`--wallpaper-command`/`SATPAPER_WALLPAPER_COMMAND` - custom command to run when a wallpaper is generated.
     - This overrides the automatic update handling.
     - Currently, this only works on Unix. The command will be run as `sh -c "{command} file://{image_path}`.

src/slider.rs

@@ -198,70 +198,6 @@ enum Direction {
     Right
 }
 
-fn cutout_disk_flood(image: &mut DynamicImage) {
-    use std::collections::VecDeque;
-
-    let x_max = image.dimensions().0 - 1;
-    let y_max = image.dimensions().1 - 1;
-
-    let step = |mut x: u32, mut y: u32, direction: Direction| {
-        use Direction::*;
-
-        match direction {
-            Up => y = y.saturating_add(1),
-            Down => y = y.saturating_sub(1),
-            Left => x = x.saturating_sub(1),
-            Right => x = x.saturating_add(1),
-        };
-
-        (x, y)
-    };
-
-    let mut flood_fill = |x: u32, y: u32| {
-        use Direction::*;
-
-        let mut queue = VecDeque::new();
-
-        queue.push_back((x, y));
-
-        while !queue.is_empty() {
-            let (x, y) = queue
-                .pop_front()
-                .expect("Queue should not be empty");
-
-            if x > x_max ||
-               y > y_max
-            {
-                continue;
-            }
-
-            let pixel = image.get_pixel(x, y);
-
-            // It turns out that some pixels in the "black" background are actually [1, 1, 1, 255]!
-            // Why? Who knows.
-            if pixel.0 <= BLACK.0 && pixel != CLEAR {
-                image.put_pixel(x, y, CLEAR);
-
-                queue.push_back(step(x, y, Up));
-                queue.push_back(step(x, y, Down));
-                queue.push_back(step(x, y, Left));
-                queue.push_back(step(x, y, Right));
-            }
-        }
-    };
-
-    // These might seem redundant, but it's necessary to handle differences in the
-    // satellite imagery - GOES has no space between the Earth and image edge on the left and right,
-    // while the rest do.
-    //
-    // This handles the "edge case" of GOES's separated corners, while not doing any extra work
-    // for the likes of Himawari (because the first fill will get all the pixels, causing the rest to finish immediately.)
-    flood_fill(0, 0);
-    flood_fill(x_max, 0);
-    flood_fill(0, y_max);
-    flood_fill(x_max, y_max);
-}
-
 // Identifies the bounds of the Earth in the image
 fn cutout_disk(image: &mut DynamicImage) {
     // Find the midpoint and max of the edges.