[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdesupport/eigen2/Eigen/src/Core
From:       BenoƮt Jacob <jacob.benoit.1 () gmail ! com>
Date:       2008-11-30 21:49:02
Message-ID: 1228081742.940998.30051.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 891007 by bjacob:

add internal documentation



 M  +2 -2      Dot.h  
 M  +3 -4      Matrix.h  
 M  +22 -4     util/Constants.h  
 M  +5 -2      util/Macros.h  
 M  +22 -0     util/XprHelper.h  


--- trunk/kdesupport/eigen2/Eigen/src/Core/Dot.h #891006:891007
@@ -273,7 +273,7 @@
 
 /** \returns the squared norm of *this, i.e. the dot product of *this with itself.
   *
-  * \note This is \em not the \em l2 norm.
+  * \note This is \em not the \em l2 norm, but its square.
   *
   * \deprecated Use squaredNorm() instead. This norm2() function is kept only for \
                compatibility and will be removed in Eigen 2.0.
   *
@@ -282,7 +282,7 @@
   * \sa dot(), norm()
   */
 template<typename Derived>
-inline typename NumTraits<typename ei_traits<Derived>::Scalar>::Real \
MatrixBase<Derived>::norm2() const +EIGEN_DEPRECATED inline typename \
NumTraits<typename ei_traits<Derived>::Scalar>::Real MatrixBase<Derived>::norm2() \
const  {
   return ei_real(dot(*this));
 }
--- trunk/kdesupport/eigen2/Eigen/src/Core/Matrix.h #891006:891007
@@ -44,11 +44,11 @@
   *
   * \note <b>Fixed-size versus dynamic-size:</b>
   * Fixed-size means that the numbers of rows and columns are known are \
                compile-time. In this case, Eigen allocates the array
-  * of coefficients as a fixed-size array on the stack. This makes sense for very \
small matrices, typically up to 4x4, sometimes up +  * of coefficients as a \
fixed-size array, as a class member. This makes sense for very small matrices, \
                typically up to 4x4, sometimes up
   * to 16x16. Larger matrices should be declared as dynamic-size even if one happens \
                to know their size at compile-time.
   *
   * Dynamic-size means that the numbers of rows or columns are not necessarily known \
                at compile-time. In this case they are runtime variables,
-  * and the array of coefficients is allocated dynamically, typically on the heap \
(See note on heap allocation below). +  * and the array of coefficients is allocated \
                dynamically, typically on the heap (See note on Usage of alloca() \
                below).
   *
   * Note that dense matrices, be they Fixed-size or Dynamic-size, <em>do not</em> \
                expand dynamically in the sense of a std::map.
   * If you want this behavior, see the Sparse module.
@@ -60,7 +60,7 @@
   * exceed a certain value. This happens when taking dynamic-size blocks inside \
                fixed-size matrices: in this case _MaxRows and _MaxCols
   * are the dimensions of the original matrix, while _Rows and _Cols are Dynamic.
   *
-  * \note <b> Heap allocation:</b>
+  * \note <b> Usage of alloca():</b>
   * On the Linux platform, for small enough arrays, Eigen will avoid heap allocation \
                and instead will use alloca() to perform a dynamic
   * allocation on the stack.
   *
@@ -85,7 +85,6 @@
   * v[1] = 0.2;
   * v(0) = 0.1;
   * v(1) = 0.2;
-
   *
   * Eigen::MatrixXi m(10, 10);
   * m(0, 1) = 1;
--- trunk/kdesupport/eigen2/Eigen/src/Core/util/Constants.h #891006:891007
@@ -26,7 +26,25 @@
 #ifndef EIGEN_CONSTANTS_H
 #define EIGEN_CONSTANTS_H
 
+/** This value means that a quantity is not known at compile-time, and that instead \
the value is +  * stored in some runtime variable.
+  *
+  * Explanation for the choice of this value:
+  * - It should be positive and larger than any reasonable compile-time-fixed number \
of rows or columns. +  *   This means that it should be at least 128 or so.
+  * - It should be smaller than the sqrt of INT_MAX. Indeed, we often multiply a \
number of rows with a number +  *   of columns in order to compute a number of \
coefficients. Even if we guard that with an "if" checking whether +  *   the values \
are Dynamic, we still get a compiler warning "integer overflow". So the only way to \
get around +  *   it would be a meta-selector. Doing this everywhere would reduce \
code readability and lenghten compilation times. +  *   Also, disabling compiler \
warnings for integer overflow, sounds like a bad idea. +  *
+  * If you wish to port Eigen to a platform where sizeof(int)==2, it is perfectly \
possible to set Dynamic to, say, 250. +  */
 const int Dynamic = 10000;
+
+/** This value means +Infinity; it is currently used only as the p parameter to \
MatrixBase::lpNorm<int>(). +  * The value Infinity there means the L-infinity norm.
+  */
 const int Infinity = -1;
 
 /** \defgroup flags flags
@@ -199,9 +217,9 @@
 };
 
 enum {
-  CompleteUnrolling,
+  NoUnrolling,
   InnerUnrolling,
-  NoUnrolling
+  CompleteUnrolling
 };
 
 enum {
@@ -211,9 +229,9 @@
 
 enum {
   IsDense         = 0,
+  IsSparse        = SparseBit,
   NoDirectAccess  = 0,
-  HasDirectAccess = DirectAccessBit,
-  IsSparse        = SparseBit
+  HasDirectAccess = DirectAccessBit
 };
 
 const int FullyCoherentAccessPattern  = 0x1;
--- trunk/kdesupport/eigen2/Eigen/src/Core/util/Macros.h #891006:891007
@@ -28,7 +28,10 @@
 
 #undef minor
 
-/** \internal  Defines the maximal loop size to enable meta unrolling of loops */
+/** \internal  Defines the maximal loop size to enable meta unrolling of loops.
+  *            Note that the value here is expressed in Eigen's own notion of \
"number of FLOPS", +  *            it does not correspond to the number of iterations \
or the number of instructions +  */
 #ifndef EIGEN_UNROLLING_LIMIT
 #define EIGEN_UNROLLING_LIMIT 100
 #endif
@@ -36,7 +39,7 @@
 /** \internal Define the maximal size in Bytes of L2 blocks.
   * The current value is set to generate blocks of 256x256 for float */
 #ifndef EIGEN_TUNE_FOR_L2_CACHE_SIZE
-#define EIGEN_TUNE_FOR_L2_CACHE_SIZE (1024*256)
+#define EIGEN_TUNE_FOR_L2_CACHE_SIZE (sizeof(float)*256*256)
 #endif
 
 #define USING_PART_OF_NAMESPACE_EIGEN \
--- trunk/kdesupport/eigen2/Eigen/src/Core/util/XprHelper.h #891006:891007
@@ -41,6 +41,10 @@
     ei_no_assignment_operator& operator=(const ei_no_assignment_operator&);
 };
 
+/** \internal If the template parameter Value is Dynamic, this class is just a \
wrapper around an int variable that +  * can be accessed using value() and \
setValue(). +  * Otherwise, this class is an empty structure and value() just returns \
the template parameter Value. +  */
 template<int Value> class ei_int_if_dynamic EIGEN_EMPTY_STRUCT
 {
   public:
@@ -136,6 +140,24 @@
 template<typename T> struct ei_must_nest_by_value { enum { ret = false }; };
 template<typename T> struct ei_must_nest_by_value<NestByValue<T> > { enum { ret = \
true }; };  
+/** \internal Determines how a given expression should be nested into another one.
+  * For example, when you do a * (b+c), Eigen will determine how the expression b+c \
should be +  * nested into the bigger product expression. The choice is between \
nesting the expression b+c as-is, or +  * evaluating that expression b+c into a \
temporary variable d, and nest d so that the resulting expression is +  * a*d. \
Evaluating can be beneficial for example if every coefficient access in the resulting \
expression causes +  * many coefficient accesses in the nested expressions -- as is \
the case with matrix product for example. +  *
+  * \param T the type of the expression being nested
+  * \param n the number of coefficient accesses in the nested expression for each \
coefficient access in the bigger expression. +  *
+  * Example. Suppose that a, b, and c are of type Matrix3d. The user forms the \
expression a*(b+c). +  * b+c is an expression "sum of matrices", which we will denote \
by S. In order to determine how to nest it, +  * the Product expression uses: \
ei_nested<S, 3>::ret, which turns out to be Matrix3d because the internal logic of +  \
* ei_nested determined that in this case it was better to evaluate the expression b+c \
into a temporary. On the other hand, +  * since a is of type Matrix3d, the Product \
expression nests it as ei_nested<Matrix3d, 3>::ret, which turns out to be +  * const \
Matrix3d&, because the internal logic of ei_nested determined that since a was \
already a matrix, there was no point +  * in copying it into another matrix.
+  */
 template<typename T, int n=1, typename EvalType = typename ei_eval<T>::type> struct \
ei_nested  {
   enum {


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic