didn't see the implementation yet, but don't we wonna have foreground_pixels and background_pixels passed in as refs?