Re: [x265] [PATCH] testbench: include intrinsics header corresponding to target architecture

2017-08-10 Thread Pradeep Ramachandran 
On Thu, Aug 10, 2017 at 10:42 AM,  wrote:

> # HG changeset patch
> # User Bhavna Hariharan 
> # Date 1502196995 -19800
> #  Tue Aug 08 18:26:35 2017 +0530
> # Node ID 4a2866922bb8241c90a1b76e4829ba0d1783097f
> # Parent  3a4f36d1501a955db00e5d181210dcc472e61529
> testbench: include intrinsics header corresponding to target architecture
>
> Display warning when dhdr10 is enabled with MSVC version that does not
> support c++11.
>

Pushed to default branch.


>
> diff -r 3a4f36d1501a -r 4a2866922bb8 source/CMakeLists.txt
> --- a/source/CMakeLists.txt Fri Jul 28 12:19:16 2017 +0530
> +++ b/source/CMakeLists.txt Tue Aug 08 18:26:35 2017 +0530
> @@ -184,6 +184,9 @@
>  endif()
>  # this option is to enable the inclusion of dynamic HDR10 library to the
> libx265 compilation
>  option(ENABLE_HDR10_PLUS "Enable dynamic HDR10 compilation" OFF)
> +if(MSVC AND (MSVC_VERSION LESS 1800) AND ENABLE_HDR10_PLUS)
> +message(FATAL_ERROR "MSVC version 12.0 or above required to support
> hdr10plus")
> +endif()
>  if(GCC)
>  add_definitions(-Wall -Wextra -Wshadow)
>  add_definitions(-D__STDC_LIMIT_MACROS=1)
> diff -r 3a4f36d1501a -r 4a2866922bb8 source/dynamicHDR10/
> metadataFromJson.cpp
> --- a/source/dynamicHDR10/metadataFromJson.cpp  Fri Jul 28 12:19:16 2017
> +0530
> +++ b/source/dynamicHDR10/metadataFromJson.cpp  Tue Aug 08 18:26:35 2017
> +0530
> @@ -33,11 +33,7 @@
>
>  #include "BasicStructures.h"
>  #include "SeiMetadataDictionary.h"
> -
> -#define M_PI 3.14159265358979323846
> -
>  using namespace SeiMetadataDictionary;
> -
>  class metadataFromJson::DynamicMetaIO
>  {
>  public:
> diff -r 3a4f36d1501a -r 4a2866922bb8 source/test/testharness.h
> --- a/source/test/testharness.h Fri Jul 28 12:19:16 2017 +0530
> +++ b/source/test/testharness.h Tue Aug 08 18:26:35 2017 +0530
> @@ -68,6 +68,10 @@
>  #include 
>  #elif HAVE_RDTSC
>  #include 
> +#elif (!defined(__APPLE__) && (defined (__GNUC__) && (defined(__x86_64__)
> || defined(__i386__
> +#include 
> +#elif ( !defined(__APPLE__) && defined (__GNUC__) &&
> defined(__ARM_NEON__))
> +#include 
>  #elif defined(__GNUC__)
>  /* fallback for older GCC/MinGW */
>  static inline uint32_t __rdtsc(void)
>
> ___
> x265-devel mailing list
> x265-devel@videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>
>
___
x265-devel mailing list
x265-devel@videolan.org
https://mailman.videolan.org/listinfo/x265-devel

Re: [x265] [PATCH 1 of 2] Introduce refine-inter levels 2 and 3

2017-08-10 Thread Pradeep Ramachandran 
On Thu, Aug 10, 2017 at 12:47 PM, Mario *LigH* Rohkrämer 
wrote:

> Am 10.08.2017, 09:16 Uhr, schrieb Mario *LigH* Rohkrämer  >:
>
>
>> Online CLI documentation does not yet contain these parameters at all:
>>
>> http://x265.readthedocs.io/en/default/cli.html?highlight=refine
>>
>
>
> Sorry, my mistake, something slipped in my browser.
>
>
readthedocs.io takes some time to update the docs. I see them there now.


>
> --
>
> Fun and success!
> Mario *LigH* Rohkrämer
> mailto:cont...@ligh.de
>
> ___
> x265-devel mailing list
> x265-devel@videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>
___
x265-devel mailing list
x265-devel@videolan.org
https://mailman.videolan.org/listinfo/x265-devel

Re: [x265] [PATCH 1 of 2] Introduce refine-inter levels 2 and 3

2017-08-10 Thread Mario *LigH* Rohkrämer 

Am 10.08.2017, 09:16 Uhr, schrieb Mario *LigH* Rohkrämer :



Online CLI documentation does not yet contain these parameters at all:

http://x265.readthedocs.io/en/default/cli.html?highlight=refine



Sorry, my mistake, something slipped in my browser.

--

Fun and success!
Mario *LigH* Rohkrämer
mailto:cont...@ligh.de

___
x265-devel mailing list
x265-devel@videolan.org
https://mailman.videolan.org/listinfo/x265-devel

Re: [x265] [PATCH 1 of 2] Introduce refine-inter levels 2 and 3

2017-08-10 Thread Mario *LigH* Rohkrämer 


Online CLI documentation does not yet contain these parameters at all:

http://x265.readthedocs.io/en/default/cli.html?highlight=refine


Am 10.08.2017, 08:50 Uhr, schrieb Pradeep Ramachandran  
:



On Wed, Aug 9, 2017 at 6:11 PM,  wrote:


# HG changeset patch
# User Bhavna Hariharan 
# Date 1500881283 -19800
#  Mon Jul 24 12:58:03 2017 +0530
# Node ID 81f17d9c7c273e4b282be66bf8bcd193e25a1d2e
# Parent  d11482e5fedbcdaf62ee3c6872f43827d99ad181
Introduce refine-inter levels 2 and 3



Pushed both patches to default branch


--

Fun and success!
Mario *LigH* Rohkrämer
mailto:cont...@ligh.de

___
x265-devel mailing list
x265-devel@videolan.org
https://mailman.videolan.org/listinfo/x265-devel

Re: [x265] [PATCH 1 of 2] Introduce refine-inter levels 2 and 3

2017-08-10 Thread Pradeep Ramachandran 
On Wed, Aug 9, 2017 at 6:11 PM,  wrote:

> # HG changeset patch
> # User Bhavna Hariharan 
> # Date 1500881283 -19800
> #  Mon Jul 24 12:58:03 2017 +0530
> # Node ID 81f17d9c7c273e4b282be66bf8bcd193e25a1d2e
> # Parent  d11482e5fedbcdaf62ee3c6872f43827d99ad181
> Introduce refine-inter levels 2 and 3
>

Pushed both patches to default branch


>
> refine-inter 2 limits the modes evaluated in the encode that uses
> --analysis-reuse-mode load.
> 2nx2n in save encode - disable re-evaluation of rect and amp
> skip in save encode  - re-evaluates only skip, merge and 2nx2n modes
>
> refine-inter 3 will force only the depth from the encode that uses
> --analysis-reuse-mode=save and re-evaluates all the modes in the
> encode that uses --analysis-reuse-mode=load.
>
> diff -r d11482e5fedb -r 81f17d9c7c27 doc/reST/cli.rst
> --- a/doc/reST/cli.rst  Mon Jul 24 11:15:38 2017 +0530
> +++ b/doc/reST/cli.rst  Mon Jul 24 12:58:03 2017 +0530
> @@ -897,24 +897,38 @@
>
> Enables refinement of intra blocks in current encode.
>
> -   Level 0 - Forces both mode and depth from the previous encode.
> +   Level 0 - Forces both mode and depth from the save encode.
>
> -   Level 1 - Evaluates all intra modes for blocks of size one smaller
> than
> -   the min-cu-size of the incoming analysis data from the previous
> encode,
> -   forces modes for blocks of larger size.
> +   Level 1 - Evaluates all intra modes at current depth(n) and at
> depth
> +   (n+1) when current block size is one greater than the min-cu-size.
> +   Forces modes for larger blocks.
>
> -   Level 2 - Evaluates all intra modes for blocks of size one smaller
> than
> -   the min-cu-size of the incoming analysis data from the previous
> encode.
> -   For larger blocks, force only depth when angular mode is chosen by
> the
> -   previous encode, force depth and mode when other intra modes are
> chosen.
> +   Level 2 - In addition to the functionality of level 1, at all
> depths, force
> +   (a) only depth when angular mode is chosen by the save encode.
> +   (b) depth and mode when other intra modes are chosen by the save
> encode.
>
> Default 0.
>
> -.. option:: --refine-inter-depth
> -
> -   Enables refinement of inter blocks in current encode. Evaluates all
> -   inter modes for blocks of size one smaller than the min-cu-size of
> the
> -   incoming analysis data from the previous encode. Default disabled.
> +.. option:: --refine-inter <0..3>
> +
> +   Enables refinement of inter blocks in current encode.
> +
> +   Level 0 - Forces both mode and depth from the save encode.
> +
> +   Level 1 - Evaluates all inter modes at current depth(n) and at
> depth
> +   (n+1) when current block size is one greater than the min-cu-size.
> +   Forces modes for larger blocks.
> +
> +   Level 2 - In addition to the functionality of level 1, restricts
> the modes
> +   evaluated when specific modes are decided as the best mode by the
> save encode.
> +
> +   2nx2n in save encode - disable re-evaluation of rect and amp.
> +
> +   skip in save encode  - re-evaluates only skip, merge and 2nx2n
> modes.
> +
> +   Level 3 - Perform analysis of inter modes while reusing depths
> from the save encode.
> +
> +   Default 0.
>
>  .. option:: --refine-mv
>
> diff -r d11482e5fedb -r 81f17d9c7c27 source/CMakeLists.txt
> --- a/source/CMakeLists.txt Mon Jul 24 11:15:38 2017 +0530
> +++ b/source/CMakeLists.txt Mon Jul 24 12:58:03 2017 +0530
> @@ -29,7 +29,7 @@
>  option(STATIC_LINK_CRT "Statically link C runtime for release builds" OFF)
>  mark_as_advanced(FPROFILE_USE FPROFILE_GENERATE NATIVE_BUILD)
>  # X265_BUILD must be incremented each time the public API is changed
> -set(X265_BUILD 131)
> +set(X265_BUILD 132)
>  configure_file("${PROJECT_SOURCE_DIR}/x265.def.in"
> "${PROJECT_BINARY_DIR}/x265.def")
>  configure_file("${PROJECT_SOURCE_DIR}/x265_config.h.in"
> diff -r d11482e5fedb -r 81f17d9c7c27 source/common/param.cpp
> --- a/source/common/param.cpp   Mon Jul 24 11:15:38 2017 +0530
> +++ b/source/common/param.cpp   Mon Jul 24 12:58:03 2017 +0530
> @@ -972,7 +972,7 @@
>  OPT("ctu-info") p->bCTUInfo = atoi(value);
>  OPT("scale-factor") p->scaleFactor = atoi(value);
>  OPT("refine-intra")p->intraRefine = atoi(value);
> -OPT("refine-inter")p->interRefine = atobool(value);
> +OPT("refine-inter")p->interRefine = atoi(value);
>  OPT("refine-mv")p->mvRefine = atobool(value);
>  OPT("force-flush")p->forceFlush = atoi(value);
>  else
> @@ -1318,6 +1318,10 @@
>  "Supported range for log2MaxPocLsb is 4 to 16");
>  CHECK(param->bCTUInfo < 0 || (param->bCTUInfo != 0 && param->bCTUInfo
> != 1 && param->bCTUInfo != 2 && param->bCTUInfo != 4 && param->bCTUInfo !=
> 6) || param->bCTUInfo > 6,
>  "Supported values