This commit moves method definitions out of a header file
into a separate compilation unit.

This commit refactors the storage and passing of entropy contexts in the
geometry octree coder.  An encapsulating class with all entropy coding
methods is used to store the context state.

The geometry coder is quite large, especially with trisoup and has no
benefit to being a header only implementation.  This commit moves the
geometry octree coder out of the header files and into
geometry_octree_{en,de}coder.cpp.

This reduces confusion when adding a unique_ptr to the node structure.

Sometimes it is useful to encode geometry without attributes.  However,
the common test condition configurations all enable attribute coding.
In order to avoid rewriting the configuration files, the
"disableAttributeCoding" option configures the encoder as if no
attributes are specified.

This commit adds configuration data for new cat1 and cat3 sequences:

cat1, voxelised frames:
  boxer_viewdep_vox12
  longdress_viewdep_vox12
  loot_viewdep_vox12
  redandblack_viewdep_vox12
  soldier_viewdep_vox12
  thaidancer_viewdep_vox12

cat3, LiDaR sqeuences:
  qnxadas-junction-approach: 74 frames
  qnxadas-junction-exit:     74 frames
  qnxadas-motorway-join:     500 frames
  qnxadas-navigating-bends:  300 frames

This commit fixes:
 - incorrect parsing of uint8 as uint16
 - incorrect address-of calculation when parsing uint16

gen-cfg.sh processes each ctc yaml file in turn, which was advantageous
when the ctc yaml files contained multiple conflicting definitions for
a test condition (eg, one defines *-lossy-attrs for colour, and another
defines *-lossy-attrs for reflectance).  In such cases, merging the
two yaml files together is not possible.

However, this does prevent having a single yaml file that modifies
the parameters for multiple test conditions. (Eg, adding sequences
to all ctc test conditions).

In order to allow extra config files to override those hard-coded
in the gen-cfg.sh script, they need to be specified at the end.

Reading and writing binary ply files is significantly faster than going
via formatted I/O.  This commit adds the --outputBinaryPly=1|0 option
to select writing either binary or ascii ply files.

Khaled Mammou authored Aug 27, 2018 and David Flynn committed Sep 03, 2018

Certain test sequences with 16-bit reflectance are actually 8-bit data that
has been scaled by 255.  This commit adds non-normative support to the
encoder and decoder to convert between the two representations through
the option "hack.reflectanceScale".

This commit disables the chroma sub-sampling in RAHT for the
common test conditions to permit better comparisons with the lifting
transform.

For some sequences, the previous scale factors (1/4 -- 1/512) are
too large for the size of the sequence.  This commit adds a procedural
determination of the scale factors from the sequence intrinsic geometry
resolution and some rules.

This configuration unifies the old configuration files and provides
greater test coverage.  Notable changes are:

 - configuration sets for octree vs trisoup, and pred/lift vs raht.

 - cat1 test sequences added to all test configurations

 - attribute quantization step sizes are aligned between all coding
   schemes.

 - levelOfDetailCount is unified where possible (cat3 = 10, mostly)

 - config dependent dist2 variation for a given sequence is removed

The dist2 values used by the pred/lift attribute coding schemes is
dependent upon the distances between points in the point cloud.  Using
the --positionQuantizationScale option to scale the input point cloud
therefore affects the required dist2 values.

This commit adds the --positionQuantizationScaleAdjustsDist2=1|0 option
to simplify the configuration of dist2.  When enabled, the encoder will
automatically scale any dist2 values by the squared position scale
factor.

 - perform expression expansion during parameter lookup

 - a list that starts with a scalar with the tag '!conditional'
   will be processed if the scalar evals to true

 - allow evaluated commands to access useful modules (hack)

This commit aligns the use of colorTransform between encoder and
decoder since this option is currently independent of what colour
space the encoded point cloud is in.

Khaled Mammou authored Aug 22, 2018 and David Flynn committed Aug 29, 2018

This commit introduces automatic derivation of dist2 values by the
frontend.  Automatic derivation occurs if fewer than levelOfDetailCount
values are provided.  No syntax changes are made.

cfg update:
 - use single dist2 values based on existing config
 - ford: 16383 -> 16384

Khaled Mammou authored Aug 23, 2018 and David Flynn committed Aug 29, 2018

This functionality is unnecessary given a uniform quant_step size.

Khaled Mammou authored Aug 22, 2018 and David Flynn committed Aug 29, 2018

This commit removes the ability to vary quant_step_size on a per-LoD
basis.  This greatly simplifies the codec configuration, especially
when used in conjunction with dist2 derivation.

cfg update:
 - rename quantizationSteps* -> quantizationStep*
 - update quantizationStep* to single values:
   - lossless-geom lossy-attrs:        8, 16, 32, 64, 128, 256 [8bit]
   - lossless-geom nearlossless-attrs: no change
   - lossless-geom lossless-attrs:     no change
   - lossy-geom    lossy-attrs:        8                       [8bit]

 => 16-bit cat3 sequences are have 8-bit qs values multipled by 255

This commit maintains consistency with other static AttributeEncoder
and AttributeDecoder methods.

Khaled Mammou authored Aug 23, 2018 and David Flynn committed Aug 29, 2018

This commit removes PCCBuildLevelOfDetail2()

Khaled Mammou authored Aug 22, 2018 and David Flynn committed Aug 29, 2018

This commit updates the hard coded limit and default values, and
updates the master configuration template.

Khaled Mammou authored Aug 22, 2018 and David Flynn committed Aug 29, 2018

This commit replaces the hard coded threshold values with APS signalling.
The default configuration maintains the hard-coded threshold values based
on bitdepth.

This option was added to use the original CfP response output position
rounding after the switch to integer point clouds.  It is not used and
its effect is barely noticeable in the objective distortion metric.
Furthermore, the encoder and decoder behaviour is different if the
option is enabled.

This is an editorial change to improve naming clarity: attr_count
represents the number of elements (dimensionality) in an attribute
set rather than the total number of attributes.

This commit adds the trisoup parameters to the GPS.

Khaled Mammou authored Aug 23, 2018 and David Flynn committed Aug 29, 2018

This commit fixes a premature determination of the point count, resulting
in no predictor weights being calculated.

Khaled Mammou authored Aug 27, 2018 and David Flynn committed Aug 29, 2018

This commit fixes an error in the hls: dist2 is capable of being a
64-bit value and must not be signalled as 32-bit.