STYLE: Rename TCoordRep template parameters to TCoordinate

Aims to make template parameter names of coordinate types clearer.

Following the ITKSoftwareGuide/CodingStyleGuide guideline on Naming Conventions,
saying, "Names are generally spelled out; use of abbreviations is discouraged".

Modules/Core/Common/include/itkBSplineInterpolationWeightFunction.h

@@ -44,17 +44,17 @@ namespace itk
  * \ingroup Functions ImageInterpolators
  * \ingroup ITKCommon
  */
-template <typename TCoordRep = float, unsigned int VSpaceDimension = 2, unsigned int VSplineOrder = 3>
+template <typename TCoordinate = float, unsigned int VSpaceDimension = 2, unsigned int VSplineOrder = 3>
 class ITK_TEMPLATE_EXPORT BSplineInterpolationWeightFunction
-  : public FunctionBase<ContinuousIndex<TCoordRep, VSpaceDimension>,
+  : public FunctionBase<ContinuousIndex<TCoordinate, VSpaceDimension>,
                         FixedArray<double, Math::UnsignedPower(VSplineOrder + 1, VSpaceDimension)>>
 {
 public:
   ITK_DISALLOW_COPY_AND_MOVE(BSplineInterpolationWeightFunction);
 
   /** Standard class type aliases. */
   using Self = BSplineInterpolationWeightFunction;
-  using Superclass = FunctionBase<ContinuousIndex<TCoordRep, VSpaceDimension>,
+  using Superclass = FunctionBase<ContinuousIndex<TCoordinate, VSpaceDimension>,
                                   FixedArray<double, Math::UnsignedPower(VSplineOrder + 1, VSpaceDimension)>>;
 
   using Pointer = SmartPointer<Self>;
@@ -83,7 +83,7 @@ class ITK_TEMPLATE_EXPORT BSplineInterpolationWeightFunction
   using SizeType = Size<VSpaceDimension>;
 
   /** ContinuousIndex type alias support. */
-  using ContinuousIndexType = ContinuousIndex<TCoordRep, VSpaceDimension>;
+  using ContinuousIndexType = ContinuousIndex<TCoordinate, VSpaceDimension>;
 
   /** The support region size: a hypercube of length SplineOrder + 1 */
   static constexpr SizeType SupportSize{ SizeType::Filled(VSplineOrder + 1) };

Modules/Core/Common/include/itkBSplineInterpolationWeightFunction.hxx

@@ -26,9 +26,9 @@
 namespace itk
 {
 /** Compute weights for interpolation at continuous index position */
-template <typename TCoordRep, unsigned int VSpaceDimension, unsigned int VSplineOrder>
+template <typename TCoordinate, unsigned int VSpaceDimension, unsigned int VSplineOrder>
 auto
-BSplineInterpolationWeightFunction<TCoordRep, VSpaceDimension, VSplineOrder>::Evaluate(
+BSplineInterpolationWeightFunction<TCoordinate, VSpaceDimension, VSplineOrder>::Evaluate(
   const ContinuousIndexType & index) const -> WeightsType
 {
   WeightsType weights;
@@ -40,9 +40,9 @@ BSplineInterpolationWeightFunction<TCoordRep, VSpaceDimension, VSplineOrder>::Ev
 }
 
 /** Compute weights for interpolation at continuous index position */
-template <typename TCoordRep, unsigned int VSpaceDimension, unsigned int VSplineOrder>
+template <typename TCoordinate, unsigned int VSpaceDimension, unsigned int VSplineOrder>
 void
-BSplineInterpolationWeightFunction<TCoordRep, VSpaceDimension, VSplineOrder>::Evaluate(
+BSplineInterpolationWeightFunction<TCoordinate, VSpaceDimension, VSplineOrder>::Evaluate(
   const ContinuousIndexType & index,
   WeightsType &               weights,
   IndexType &                 startIndex) const

Modules/Core/Common/include/itkBoundingBox.h

@@ -55,7 +55,7 @@ namespace itk
  * Template parameters for BoundingBox:
  *
  * \tparam TPointIdentifier The type used to access a particular point (i.e., a point's id)
- * \tparam TCoordRep Numerical type with which to represent each coordinate value.
+ * \tparam TCoordinate Numerical type with which to represent each coordinate value.
  * \tparam VPointDimension Geometric dimension of space.
  *
  * \ingroup DataRepresentation
@@ -65,8 +65,8 @@ namespace itk
  */
 template <typename TPointIdentifier = IdentifierType,
           unsigned int VPointDimension = 3,
-          typename TCoordRep = float,
-          typename TPointsContainer = VectorContainer<TPointIdentifier, Point<TCoordRep, VPointDimension>>>
+          typename TCoordinate = float,
+          typename TPointsContainer = VectorContainer<TPointIdentifier, Point<TCoordinate, VPointDimension>>>
 class ITK_TEMPLATE_EXPORT BoundingBox : public Object
 {
 public:
@@ -89,7 +89,7 @@ class ITK_TEMPLATE_EXPORT BoundingBox : public Object
 
   /** Hold on to the type information specified by the template parameters. */
   using PointIdentifier = TPointIdentifier;
-  using CoordRepType = TCoordRep;
+  using CoordRepType = TCoordinate;
   using PointsContainer = TPointsContainer;
   using PointsContainerPointer = typename PointsContainer::Pointer;
   using PointsContainerConstPointer = typename PointsContainer::ConstPointer;

Modules/Core/Common/include/itkBoundingBox.hxx

@@ -35,10 +35,10 @@ namespace itk
 /**
  * Print out the bounding box.
  */
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 void
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::PrintSelf(std::ostream & os,
-                                                                                       Indent         indent) const
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::PrintSelf(std::ostream & os,
+                                                                                         Indent         indent) const
 {
   Superclass::PrintSelf(os, indent);
 
@@ -53,9 +53,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Pri
 /**
  * Access routine to set the points container.
  */
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 void
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::SetPoints(const PointsContainer * points)
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::SetPoints(const PointsContainer * points)
 {
   itkDebugMacro("setting Points container to " << points);
   if (m_PointsContainer != points)
@@ -66,9 +66,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Set
 }
 
 /** Access routine to get the points container. */
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 auto
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::GetPoints() const
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::GetPoints() const
   -> const PointsContainer *
 {
   itkDebugMacro("returning Points container of " << m_PointsContainer);
@@ -77,9 +77,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Get
 }
 
 /** Compute and get the corners of the bounding box */
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 auto
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::ComputeCorners() const
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::ComputeCorners() const
   -> std::array<PointType, NumberOfCorners>
 {
   std::array<PointType, NumberOfCorners> result;
@@ -110,9 +110,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Com
 
 #if !defined(ITK_LEGACY_REMOVE)
 /** Compute and get the corners of the bounding box */
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 auto
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::GetCorners() -> const PointsContainer *
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::GetCorners() -> const PointsContainer *
 {
   m_CornersContainer->clear();
   m_CornersContainer->Reserve(NumberOfCorners);
@@ -128,9 +128,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Get
 #endif
 
 
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 bool
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::ComputeBoundingBox() const
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::ComputeBoundingBox() const
 {
   if (!m_PointsContainer)
   {
@@ -153,8 +153,8 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Com
       return false;
     }
 
-    PointsContainerConstIterator      ci = m_PointsContainer->Begin();
-    Point<TCoordRep, VPointDimension> point = ci->Value(); // point value
+    PointsContainerConstIterator        ci = m_PointsContainer->Begin();
+    Point<TCoordinate, VPointDimension> point = ci->Value(); // point value
     for (unsigned int i = 0; i < PointDimension; ++i)
     {
       m_Bounds[2 * i] = point[i];
@@ -187,9 +187,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Com
   return true;
 }
 
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 auto
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::GetCenter() const -> PointType
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::GetCenter() const -> PointType
 {
   this->ComputeBoundingBox();
 
@@ -202,9 +202,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Get
   return center;
 }
 
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 auto
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::GetMinimum() const -> PointType
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::GetMinimum() const -> PointType
 {
   this->ComputeBoundingBox();
 
@@ -217,9 +217,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Get
   return minimum;
 }
 
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 void
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::SetMinimum(const PointType & point)
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::SetMinimum(const PointType & point)
 {
   for (unsigned int i = 0; i < PointDimension; ++i)
   {
@@ -229,9 +229,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Set
   m_BoundsMTime.Modified();
 }
 
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 auto
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::GetMaximum() const -> PointType
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::GetMaximum() const -> PointType
 {
   this->ComputeBoundingBox();
 
@@ -244,9 +244,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Get
   return maximum;
 }
 
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 void
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::SetMaximum(const PointType & point)
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::SetMaximum(const PointType & point)
 {
   for (unsigned int i = 0; i < PointDimension; ++i)
   {
@@ -256,9 +256,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Set
   m_BoundsMTime.Modified();
 }
 
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 void
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::ConsiderPoint(const PointType & point)
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::ConsiderPoint(const PointType & point)
 {
   bool changed = false;
 
@@ -282,9 +282,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Con
   }
 }
 
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 auto
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::GetDiagonalLength2() const
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::GetDiagonalLength2() const
   -> AccumulateType
 {
   typename NumericTraits<CoordRepType>::AccumulateType dist2 = CoordRepType{};
@@ -300,9 +300,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Get
   return dist2;
 }
 
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 bool
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::IsInside(const PointType & point) const
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::IsInside(const PointType & point) const
 {
   unsigned int j = 0;
   unsigned int i = 0;
@@ -322,9 +322,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::IsI
   return true;
 }
 
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 ModifiedTimeType
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::GetMTime() const
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::GetMTime() const
 {
   ModifiedTimeType latestTime = Object::GetMTime();
 
@@ -335,9 +335,9 @@ BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::Get
   return latestTime;
 }
 
-template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordRep, typename TPointsContainer>
+template <typename TPointIdentifier, unsigned int VPointDimension, typename TCoordinate, typename TPointsContainer>
 auto
-BoundingBox<TPointIdentifier, VPointDimension, TCoordRep, TPointsContainer>::DeepCopy() const -> Pointer
+BoundingBox<TPointIdentifier, VPointDimension, TCoordinate, TPointsContainer>::DeepCopy() const -> Pointer
 {
   Pointer clone = Self::New();
 

Modules/Core/Common/include/itkCellInterface.h

@@ -513,7 +513,7 @@ class ITK_TEMPLATE_EXPORT CellInterface
  * \ingroup ITKCommon
  */
 template <int VPointDimension,
-          typename TCoordRep,
+          typename TCoordinate,
           typename TInterpolationWeight,
           typename TPointIdentifier,
           typename TCellIdentifier,
@@ -525,7 +525,7 @@ class ITK_TEMPLATE_EXPORT CellTraitsInfo
 {
 public:
   static constexpr unsigned int PointDimension = VPointDimension;
-  using CoordRepType = TCoordRep;
+  using CoordRepType = TCoordinate;
   using InterpolationWeightType = TInterpolationWeight;
   using PointIdentifier = TPointIdentifier;
   using CellIdentifier = TCellIdentifier;

Modules/Core/Common/include/itkContinuousIndex.h

@@ -30,7 +30,7 @@ namespace itk
  *
  * ContinuousIndex is a templated class that holds a set of coordinates
  * (components).
- * The template parameter TCoordRep can be any floating point type (float, double).
+ * The template parameter TCoordinate can be any floating point type (float, double).
  * The VIndexDimension defines the number of  components in the continuous
  * index array.
  *
@@ -42,21 +42,21 @@ namespace itk
  *
  * \ingroup ITKCommon
  */
-template <typename TCoordRep = double, unsigned int VIndexDimension = 2>
-class ITK_TEMPLATE_EXPORT ContinuousIndex : public Point<TCoordRep, VIndexDimension>
+template <typename TCoordinate = double, unsigned int VIndexDimension = 2>
+class ITK_TEMPLATE_EXPORT ContinuousIndex : public Point<TCoordinate, VIndexDimension>
 {
-  static_assert(std::is_floating_point_v<TCoordRep>,
+  static_assert(std::is_floating_point_v<TCoordinate>,
                 "The coordinates of a continuous index must be represented by floating point numbers.");
 
 public:
   /** Standard class type aliases. */
   using Self = ContinuousIndex;
-  using Superclass = Point<TCoordRep, VIndexDimension>;
+  using Superclass = Point<TCoordinate, VIndexDimension>;
 
   /** ValueType can be used to declare a variable that is the same type
    * as a data element held in an Point.   */
-  using ValueType = TCoordRep;
-  using CoordRepType = TCoordRep;
+  using ValueType = TCoordinate;
+  using CoordRepType = TCoordinate;
 
   /** Dimension of the Space */
   static constexpr unsigned int IndexDimension = VIndexDimension;
@@ -83,7 +83,7 @@ class ITK_TEMPLATE_EXPORT ContinuousIndex : public Point<TCoordRep, VIndexDimens
   {
     for (unsigned int i = 0; i < VIndexDimension; ++i)
     {
-      (*this)[i] = static_cast<TCoordRep>(index[i]);
+      (*this)[i] = static_cast<TCoordinate>(index[i]);
     }
   }
 };

Modules/Core/Common/include/itkCovariantVector.h

@@ -245,9 +245,9 @@ class ITK_TEMPLATE_EXPORT CovariantVector : public FixedArray<T, VVectorDimensio
 
   /** Copy from another CovariantVector with a different representation type.
    *  Casting is done with C-Like rules  */
-  template <typename TCoordRepB>
+  template <typename TCoordinateB>
   void
-  CastFrom(const CovariantVector<TCoordRepB, VVectorDimension> & pa)
+  CastFrom(const CovariantVector<TCoordinateB, VVectorDimension> & pa)
   {
     for (unsigned int i = 0; i < VVectorDimension; ++i)
     {

Modules/Core/Common/include/itkDefaultDynamicMeshTraits.h

@@ -46,7 +46,7 @@ namespace itk
  * \tparam VMaxTopologicalDimension Max topological dimension of a cell
  * that can be inserted into this mesh.
  *
- * \tparam TCoordRep Numerical type to store each coordinate value.
+ * \tparam TCoordinate Numerical type to store each coordinate value.
  *
  * \tparam TInterpolationWeight Numerical type to store interpolation
  * weights.
@@ -57,7 +57,7 @@ namespace itk
 template <typename TPixelType,
           unsigned int VPointDimension = 3,
           unsigned int VMaxTopologicalDimension = VPointDimension,
-          typename TCoordRep = float,
+          typename TCoordinate = float,
           typename TInterpolationWeight = float,
           typename TCellPixelType = TPixelType>
 class DefaultDynamicMeshTraits
@@ -69,7 +69,7 @@ class DefaultDynamicMeshTraits
   /** Just save all the template parameters. */
   using PixelType = TPixelType;
   using CellPixelType = TCellPixelType;
-  using CoordRepType = TCoordRep;
+  using CoordRepType = TCoordinate;
   using InterpolationWeightType = TInterpolationWeight;
 
   /** Just save all the template parameters. */