biotrump / libyuv

Automatically exported from code.google.com/p/libyuv
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Arm unittest failures #479

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
3 unittests are failing on arm:

[  FAILED  ] libyuvTest.TestJ420
Likely its just the test, because the actual I420ToARGB passed.

[  FAILED  ] libyuvTest.ARGBSobelToPlane_Any
The odd width support must be wrong?

[  FAILED  ] libyuvTest.TestScaleJ420
Same as TestI420.  This test doesnt actually do anything useful yet, so I'll 
disable.

Original issue reported on code.google.com by fbarch...@chromium.org on 28 Jul 2015 at 12:29

GoogleCodeExporter commented 9 years ago
r1454 fixes libyuvTest.ARGBSobelToPlane_Any and disables TestScaleJ420

Original comment by fbarch...@chromium.org on 28 Jul 2015 at 11:05

GoogleCodeExporter commented 9 years ago
Attached is full output
[ RUN      ] libyuvTest.ARGBSobelToPlane_Any
../../unit_test/planar_test.cc:1691: Failure
Value of: max_diff
  Actual: 255
Expected: 0
[  FAILED  ] libyuvTest.ARGBSobelToPlane_Any (2 ms)

../../unit_test/color_test.cc:134: Failure
The difference between static_cast<int>(orig_pixels[i]) and 
static_cast<int>(dst_pixels_opt[i]) is 4, which exceeds 3, where
static_cast<int>(orig_pixels[i]) evaluates to 0,
static_cast<int>(dst_pixels_opt[i]) evaluates to 4, and
3 evaluates to 3.
[  FAILED  ] libyuvTest.TestJ420 (86 ms)

Original comment by fbarch...@chromium.org on 7 Aug 2015 at 5:47

Attachments:

GoogleCodeExporter commented 9 years ago
fixed in r1461.  increased arm error tolerance to 4 from 3.

Original comment by fbarch...@chromium.org on 7 Aug 2015 at 7:32

GoogleCodeExporter commented 9 years ago

Original comment by fbarch...@chromium.org on 7 Aug 2015 at 7:33