summaryrefslogtreecommitdiff
path: root/vp8/encoder
AgeCommit message (Collapse)Author
2012-10-23Merge "Change eob[] array type in splitmv RD loop to a regular integer." ↵pascal massimino
into experimental
2012-10-23Change eob[] array type in splitmv RD loop to a regular integer.Ronald S. Bultje
Change-Id: I240d6b50069fd3f35cc4fed2f4507796f0ef25e9
2012-10-23Merging in the Switchable interp experimentDeb Mukherjee
There is a macro DEFAULT_INTERP_FILTER defined in encoder/onyx_if.c that is set as EIGHTTAP for now - so SWITCHABLE is not really used. Ideally, this should be SWITCHABLE but that would make the encoder quite a bit slower. We will change the default filter to SWITCHABLE once we find a faster way to search for switchable filters. Change-Id: Iee91832cdc07e6e14108d9b543130fdd12fc9874
2012-10-22Merge "Change entropy context assignment style from *(x + y) = z to x[y] = ↵Ronald S. Bultje
z." into experimental
2012-10-22Merging in the TX_SELECT experimentDeb Mukherjee
Change-Id: I0d306b0062be59bc785b65882691c956c8b5e7c8
2012-10-22Merge of the newmventropy experimentDeb Mukherjee
Removes the old mv encoding code. Change-Id: I1026d48cc5ac15ccb776f98463e929254c3dc7da
2012-10-22Change entropy context assignment style from *(x + y) = z to x[y] = z.Ronald S. Bultje
Change-Id: Ibc4425bb6e01606751021e57c93106c365a2da21
2012-10-22Merge "Added rtcd support vp8_sad16x3 and vp8_sad3x16" into experimentalScott LaVarnway
2012-10-22Added rtcd support vp8_sad16x3 and vp8_sad3x16Scott LaVarnway
Change-Id: I5bca7b7a4b230082d36ac6fb84db84137ad177d7
2012-10-22Merge changes I02e7f64a,Ide954b00,Idc8b5977 into experimentalRonald S. Bultje
* changes: Fix another typo in 4x4-transform-for-i8x8-intra-pred coeff contexts. 8x8 transform support in splitmv. Use SPLITMV_PARTITIONING instead of a plain integer type.
2012-10-22Merge "Merging the hybrid transform experiments" into experimentalDeb Mukherjee
2012-10-22Merging the hybrid transform experimentsDeb Mukherjee
Change-Id: I99f1982b30a630a9a070a8326d83b34a33cba14c
2012-10-22Fix another typo in 4x4-transform-for-i8x8-intra-pred coeff contexts.Ronald S. Bultje
Change-Id: I02e7f64a7c99f6c222b9ef21341b0e801d646810
2012-10-228x8 transform support in splitmv.Ronald S. Bultje
For splitmv, where partitioning is 8x16, 16x8 or 8x8, this patch uses the 8x8 transform (instead of the 4x4) if txfm_mode is ALLOW_8X8 or ALLOW_16X16. For TX_MODE_SELECT, splitmv can indicate which of the 2 transform sizes (4x4 or 8x8) it wants to use. Gains (with hybridtx4x4/8x8/16x16 and tx_select experiments enabled) on derf: +0.9%, HD: +0.4%, STD/HD: +0.8% (SSIM or overall PSNR, both metrics show similar improvements). Change-Id: Ide954b000b415548ed92a7ac78e24f36e60fcf06
2012-10-22Use SPLITMV_PARTITIONING instead of a plain integer type.Ronald S. Bultje
This can be used to distinguish between 16x8, 8x16, 8x8 and 4x4 partitioning modes when choosing splitmv as a MB mode. Change-Id: Idc8b59772e1a80ccc4ad44d63c5c2ec3fc061a3c
2012-10-22Merge "Invert SB probability." into experimentalRonald S. Bultje
2012-10-22Merge "Remove variance vtable from rtcd" into experimentalJim Bankoski
2012-10-22Invert SB probability.Ronald S. Bultje
It currently counts the probability that the branch is true, but it should count the probability that the branch is false. Change-Id: I963825da2e7a7ed3a613eb23ffd085e427dc36e5
2012-10-22Merge "Fix use of wrong context in i8x8 coefficient cost calculation." into ↵Ronald S. Bultje
experimental
2012-10-21Remove variance vtable from rtcdJim Bankoski
Change-Id: Idd2722a538423b451e1e3495f89a7141480493d6
2012-10-21Merge "Allow B_VL_PRED & B_LD_PRED modes with Superblocks" into experimentalDeb Mukherjee
2012-10-20Allow B_VL_PRED & B_LD_PRED modes with SuperblocksDeb Mukherjee
Allows B_VL_PRED & B_LD_PRED modes to be used for all blocks within a MB in B_PRED mode. These modes were temporarily disabled with super-block coding. Change-Id: I973b9bdb82c3da5f12d7cc963162a28805f25303
2012-10-19Fix use of wrong context in i8x8 coefficient cost calculation.Ronald S. Bultje
Change-Id: Ica58d0c7d8addb5ddd5d6e1fdea8f0f96e110862
2012-10-19calculate probs consistentlyJohn Koleszar
There were several different methods for calculating bitstream probabilities in use. Consolodate these into a pair of functions, get_prob() and get_binary_prob(). Change-Id: I5534f517f74027fee16d89c9baefaafea8156b2f
2012-10-19Remove bc, bc2 from pbi,cpi,xdJohn Koleszar
Pass the bool coder to be used explicitly. This avoids cases where two different bool coders can be addressed from the same function. Also be more consistent with bool coder variable naming, start to standardize on 'bc'. Change-Id: I1c95e2fdbe24ebe8c0f84924daa1728e3b054a31
2012-10-19Some cleanups and fixes.Deb Mukherjee
Separates the logic on transform type selection previously spread out over a number of files into a separate function. Currently the tx_type field in b_mode_info is not used, but still left in there to eventually use for signaling the transform type in the bitstream. Also, now for tx_type = DCT_DCT, the regular integer DCT is used, as opposed to the floating point DCT used in conjuction with hybrid transform. Results change somewhat due to the transform change, but are within reasonable limits. The hd/std-hd sets are slightly up, while derf/yt are slightly down. Change-Id: I5776840c2239ca2da31ca6cfd7fd1148dc5f9e0f
2012-10-18Merge "Fix encode breakout skip handling." into experimentalRonald S. Bultje
2012-10-18Merge "Detect skippable blocks in splitmv RD loop case." into experimentalRonald S. Bultje
2012-10-18Merge "Use correct context for optimize_b() in vp8_optimize_mby_8x8()." into ↵Ronald S. Bultje
experimental
2012-10-18A fix for crashes when #macroblock rows is odd.Deb Mukherjee
Change-Id: If9fc2df4051ccfb8d804dae9d1be08d12d0f81ce
2012-10-17Fix encode breakout skip handling.Ronald S. Bultje
Change-Id: Ib42a5dbded27abf2a01fd656a47a1e96b1fe2912
2012-10-17Detect skippable blocks in splitmv RD loop case.Ronald S. Bultje
Change-Id: Id8a7265dc721e5cdcaa144e0041beeb32a98f51c
2012-10-17Merge "cosmetic: fix '= & ' construct" into experimentalpascal massimino
2012-10-17Merge "consolidate update_mb_segmentation_map data" into experimentalJohn Koleszar
2012-10-17Merge changes Ic711a378,I43a46c36,If19ee74a into experimentalJohn Koleszar
* changes: Move remaining per-frame data into partition 0 Interleave modes/residual per macroblock Force interleaved decoding
2012-10-17cosmetic: fix '= & ' constructJohn Koleszar
remove useless space after address-of operator. Change-Id: I1fb9e82e8d6cf87558fbd454fb5c0f87599ca2ab
2012-10-17Move remaining per-frame data into partition 0John Koleszar
This commit moves a bit of data that ended up packed with the modes/mv/residual partition during the change to interleaved encoding into partition 0 where it belongs. Change-Id: Ic711a378c58d9d6a17254384f492c213a15bad92
2012-10-17Use correct context for optimize_b() in vp8_optimize_mby_8x8().Ronald S. Bultje
Change-Id: Ic975ab85a9924adc7b3d421f64155cc79f40ffd0
2012-10-17Interleave modes/residual per macroblockJohn Koleszar
Packs the bitstream with each mb's residual following its mode/mv information. TODO: There are still a few fields that should be packed into partition 0 but are included in partition 1, due to them being serialized from write_kfmodes/pack_inter_mode_mvs, which execute after the first partition is finalized. These need to be separated out into a separate function, similar to mb_mode_mv_init() in decodemv.c. Change-Id: I43a46c363601ab36954d07ebe498760e1e2e3af4
2012-10-17Use 8x8 block bases for non-4x4 splitmv RD.Ronald S. Bultje
Change-Id: I76095d5a02edcc5d404ea830157780b24fd509d7
2012-10-16Merge "removed the recon rtcd invoke macro code (unrevert)" into experimentalJim Bankoski
2012-10-16Fix a mismatch if a keyframe contains skip-blocks.Ronald S. Bultje
Change-Id: I0681d3183f51627be8c2bb76f343b7270f9116d8
2012-10-16removed the recon rtcd invoke macro code (unrevert)Jim Bankoski
This reinstates reverted commit 2113a831575d81faeadd9966e256d58b6b2b1633 Change-Id: I9a9af13497d1e58d4f467e3e083fddf06b1b786c
2012-10-15Remove test from B_MODE_INFO.Ronald S. Bultje
It is essentially a duplicate of mode for RD-only purposes. Removing it saves us 4 bytes per B_MODE_INFO, or ~0.5MB for a 1080p video encode. Change-Id: I0a54db5f51658b3946d7efb1ca6e8cfbda0cdf88
2012-10-15Remove mode_rdopt from MB_MODE_INFO.Ronald S. Bultje
The variable is essentially a duplicate of mode for RD-only purposes. Removing it gives identical results, and saves 4 bytes per macroblock (i.e. 32.5kB for a 1080p HD video encode). Change-Id: I22d5058fdb80ab0b69862caee825e9d86bb148b3
2012-10-15Properly use chroma block object for chroma coefficient costing.Ronald S. Bultje
Change-Id: Ieb38c7aae91dbaca4a8add204fa84e1cfc459933
2012-10-15Add a new token stuffing function vp8_stuff_mb().Ronald S. Bultje
This way a caller doesn't need to implement the logic for which (and how many) tokens to write out to stuff one macroblock worth of EOBs. Make the actual function implementations static, since they are now only used in tokenize.c; also do some minor stylistic changes so it follows the style guide a little more closely; use PLANE_TYPE where appropriate, remove old (stale) frame_type function arguments; hardcode plane type where only a single one is possible (2nd order DC or U/V EOB stuffing); support stuffing 8x8/4x4 transform EOBs with no 2nd order DC. Change-Id: Ia448e251d19a4e3182eddeb9edd034bd7dc16fa3
2012-10-15Add and consistently use PLANE_TYPE.Ronald S. Bultje
Change the macros PLANE_TYPE_{Y_NO_DC,Y2,UV,Y_WITH_DC} to a typed enum, and use this typed enum consistently across all places where relevant. In places where the type is implied (e.g. in functions that only handle second order planes or chroma planes), remove it as a function argument and instead hardcode the proper enum in the code directly. Change-Id: I93652b4a36aa43163d49c732b0bf5c4442738c47
2012-10-15Merge duplicate loops in tokenization code.Ronald S. Bultje
Also merge the three occurrences of 4x4 chroma block writing into a single function, and call that function instead of duplicating the 4x4 chroma tokenization code in 3 places. Change-Id: I7913538d1029f709b0e3ae49fff1148d3be9eeb9
2012-10-14Minor refactoring in encodeintra.c.Ronald S. Bultje
Merge code blocks for different transform sizes; use MACROBLOCKD as a temp variable where that leads to smaller overall source code; remove duplicate code under #if CONFIG_HYBRIDTRANSFORM/#else blocks. Some style changes to make it follow the style guide a little better. Change-Id: I1870a06dae298243db46e14c6729c96c66196525