search

ericmckean / webm

Automatically exported from code.google.com/p/webm
0 stars 0 forks source link

test_new_vs_old_temp_scale fail #448

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Observed for
------------
WebM Project VP8 Encoder v1.1.0-31-g51fe916
Test Machine is Running: 32 bit Linux GCC

Tester Command Line
-------------------
./VP8_Tester_API_32Bit.exe  test_new_vs_old_temp_scale 
../TestClips/BBB_720x480_2000F.ivf 2 400 600 1000 0 0 
VP8vOldest_PlugIn_RLib_RMode_32Bit.exe webm

Summary Results:
----------------
-------------------------COMMIT-PSNR-LOG-------------------------

51fe916 Merge "Fix missing param name in 25.4061 psnr 385.448 kb/s 
                                         28.4869 psnr 582.068 kb/s 
                                         36.6863 psnr 988.795 kb/s 
                                         28798235 ms <--
eec30c8 Merge "fix warnings for building
871bd23 Allow target autodetection to wo
92963df fix warnings for building on win
44d35f7 Merge branch 'origin/eider' into
c7ca380 Merge "vp8e - boolcoder unit tes
297dc90 Update CHANGELOG for v1.1.0 (Eid
499510f Update AUTHORS                  
2d6cb34 Update .mailmap                 
c8f4c18 Use consistent range for VP8E_SE
9851486 vp8e - boolcoder unit test      
14d827f fix vp8_ namespace issues       
22f56b9 Formalize encodeframe.c forward 
3e32105 Fix multi-resolution threaded en
2bf8fb5 remove deprecated pre-v0.9.0 API
f039a85 Make global data const          
f86993f Merge "vp8_multi_resolution_enco 25.417 psnr 386.638 kb/s 
                                         28.5378 psnr 583.716 kb/s 
                                         38.7019 psnr 990.144 kb/s 
                                         18189767 ms
357800e Fix multi-resolution threaded en
9f9cc8f Merge "Add VPX_TS_ prefix to MAX
ef8ba4f vp8_multi_resolution_encoder.c i
9260aa8 Fix missing param name in NEON s
25a36d6 multi-res: restore v1.0.0 API    25.417 psnr 386.638 kb/s 
                                         28.5378 psnr 583.716 kb/s 
                                         38.7019 psnr 990.144 kb/s 
                                         18849430 ms
d8216b1 Merge "Fix compiler warnings" in
d46ddd0 Add VPX_TS_ prefix to MAX_LAYERS
0cc044f update unit tests to support --e
8b1a14d Add support for native Solaris c
e50c842 Fix TEXTRELs in the ARM asm.    
22ae140 Fix trailing commas in enums.   
28f5451 Fix trailing commas in enums.   
14c9fce Fix compiler warnings           

-----------------------------------------------------------------

Results:

 * New PSNR: 1 25.4061 is with in 1% of Old PSNR: 1 25.4170 - Passed
 * New PSNR: 2 28.4869 is with in 1% of Old PSNR: 2 28.5378 - Passed
 * New PSNR: 3 36.6863 <  Old PSNR: 3 38.7019 - Failed
 * New time: 28798235 > Old time: 18189767 - Failed

Failed

Other Notes:
------------
The problem seems to bisect back to here:
https://gerrit.chromium.org/gerrit/#patch,sidebyside,22153,2,vp8/vp8_cx_iface.c

Original issue reported on code.google.com by jamesbe...@google.com on 18 May 2012 at 2:37

GoogleCodeExporter commented 9 years ago 
Noise sensitivity is not supported with temporal scalability yet (see issue 
#447). vpxt_compress_scalable_patterns() should be modified to not set 
VP8E_SET_NOISE_SENSITIVITY.

Original comment by jkoles...@google.com on 18 May 2012 at 8:42

GoogleCodeExporter commented 9 years ago 
removing "vpx_codec_control (&codec_enc, VP8E_SET_NOISE_SENSITIVITY,       2);"
seems to resolve the issue.  

fix submitted:
https://gerrit.chromium.org/gerrit/#change,23065

Original comment by jamesbe...@google.com on 18 May 2012 at 9:06