Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved runtime of color bleeding from O(n^2) to O(n) #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Commits on Aug 2, 2013

  1. Improved runtime of color bleeding from O(n^2) to O(n)

    The speedup brings the runtime down from several minutes to a few seconds.
    
    Instead of starting over in each iteration an efficient breadth-first search is done. Initially, only those fully transparent pixels are added to "pending" that have at least one neighbor which is not a fully transparent pixel. This is called the "border". This border is then subsequently extended/shifted in each iteration such that only relevant pixels are considered. In the old algorithm pixels that had only fully transparent neighbors were considered again and again in each iteration until the border finally reached those pixels.
    
    Another optimization was done related to reading/writing the rgb array of BufferedImage.
    letmaik committed Aug 2, 2013
    Configuration menu
    Copy the full SHA
    e09709c View commit details
    Browse the repository at this point in the history

Commits on Aug 4, 2013

  1. Improved constant runtime of mask initialization

    When a fully transparent pixel only has fully transparent pixels around it, then this information is saved into a boolean and reused for the following pixel. This allows to reduce the number of lookups of bordering pixels to 3 (the 3 right ones) instead of all 8.
    
    On my machine around 100ms got saved with this method for a texture that had a lot of fully transparent pixels.
    letmaik committed Aug 4, 2013
    Configuration menu
    Copy the full SHA
    f98083a View commit details
    Browse the repository at this point in the history
  2. Removed unnecessary check

    The iteration only loops pixels which have at least one non-fully-transparent pixel.
    letmaik committed Aug 4, 2013
    Configuration menu
    Copy the full SHA
    9f3cbdb View commit details
    Browse the repository at this point in the history