c9b511f0e6
Reason for revert: Caused https://code.google.com/p/skia/issues/detail?id=2423 ('"Deadline exceeded" error when connecting to live rebaseline_server'). Reverting until I can figure it out. Original issue's description: > rebaseline_server: multithreaded loading/diffing of images > > BUG=skia:2414 > NOTRY=True > > Committed: http://code.google.com/p/skia/source/detail?r=14184 R=rmistry@google.com TBR=rmistry@google.com NOTREECHECKS=true NOTRY=true BUG=skia:2414 Author: epoger@google.com Review URL: https://codereview.chromium.org/239623002 git-svn-id: http://skia.googlecode.com/svn/trunk@14207 2bbb7eff-a529-9590-31e7-b0007b416f81
90 lines
3.4 KiB
Python
90 lines
3.4 KiB
Python
#!/usr/bin/python
|
|
|
|
"""
|
|
Copyright 2014 Google Inc.
|
|
|
|
Use of this source code is governed by a BSD-style license that can be
|
|
found in the LICENSE file.
|
|
|
|
ImagePair class (see class docstring for details)
|
|
"""
|
|
|
|
import posixpath
|
|
|
|
# Keys used within ImagePair dictionary representations.
|
|
# NOTE: Keep these in sync with static/constants.js
|
|
KEY__DIFFERENCE_DATA = 'differenceData'
|
|
KEY__EXPECTATIONS_DATA = 'expectations'
|
|
KEY__EXTRA_COLUMN_VALUES = 'extraColumns'
|
|
KEY__IMAGE_A_URL = 'imageAUrl'
|
|
KEY__IMAGE_B_URL = 'imageBUrl'
|
|
KEY__IS_DIFFERENT = 'isDifferent'
|
|
|
|
|
|
class ImagePair(object):
|
|
"""Describes a pair of images, pixel difference info, and optional metadata.
|
|
"""
|
|
|
|
def __init__(self, image_diff_db,
|
|
base_url, imageA_relative_url, imageB_relative_url,
|
|
expectations=None, extra_columns=None):
|
|
"""
|
|
Args:
|
|
image_diff_db: ImageDiffDB instance we use to generate/store image diffs
|
|
base_url: base of all image URLs
|
|
imageA_relative_url: string; URL pointing at an image, relative to
|
|
base_url; or None, if this image is missing
|
|
imageB_relative_url: string; URL pointing at an image, relative to
|
|
base_url; or None, if this image is missing
|
|
expectations: optional dictionary containing expectations-specific
|
|
metadata (ignore-failure, bug numbers, etc.)
|
|
extra_columns: optional dictionary containing more metadata (test name,
|
|
builder name, etc.)
|
|
"""
|
|
self.base_url = base_url
|
|
self.imageA_relative_url = imageA_relative_url
|
|
self.imageB_relative_url = imageB_relative_url
|
|
self.expectations_dict = expectations
|
|
self.extra_columns_dict = extra_columns
|
|
if not imageA_relative_url or not imageB_relative_url:
|
|
self._is_different = True
|
|
self.diff_record = None
|
|
elif imageA_relative_url == imageB_relative_url:
|
|
self._is_different = False
|
|
self.diff_record = None
|
|
else:
|
|
# TODO(epoger): Rather than blocking until image_diff_db can read in
|
|
# the image pair and generate diffs, it would be better to do it
|
|
# asynchronously: tell image_diff_db to download a bunch of file pairs,
|
|
# and only block later if we're still waiting for diff_records to come
|
|
# back.
|
|
self._is_different = True
|
|
image_diff_db.add_image_pair(
|
|
expected_image_locator=imageA_relative_url,
|
|
expected_image_url=posixpath.join(base_url, imageA_relative_url),
|
|
actual_image_locator=imageB_relative_url,
|
|
actual_image_url=posixpath.join(base_url, imageB_relative_url))
|
|
self.diff_record = image_diff_db.get_diff_record(
|
|
expected_image_locator=imageA_relative_url,
|
|
actual_image_locator=imageB_relative_url)
|
|
if self.diff_record and self.diff_record.get_num_pixels_differing() == 0:
|
|
self._is_different = False
|
|
|
|
def as_dict(self):
|
|
"""Returns a dictionary describing this ImagePair.
|
|
|
|
Uses the KEY__* constants as keys.
|
|
"""
|
|
asdict = {
|
|
KEY__IMAGE_A_URL: self.imageA_relative_url,
|
|
KEY__IMAGE_B_URL: self.imageB_relative_url,
|
|
}
|
|
asdict[KEY__IS_DIFFERENT] = self._is_different
|
|
if self.expectations_dict:
|
|
asdict[KEY__EXPECTATIONS_DATA] = self.expectations_dict
|
|
if self.extra_columns_dict:
|
|
asdict[KEY__EXTRA_COLUMN_VALUES] = self.extra_columns_dict
|
|
if self.diff_record and (self.diff_record.get_num_pixels_differing() > 0):
|
|
asdict[KEY__DIFFERENCE_DATA] = self.diff_record.as_dict()
|
|
return asdict
|