Skip to content
Snippets Groups Projects
Commit d99f321a authored by Oliver Sander's avatar Oliver Sander Committed by sander@FU-BERLIN.DE
Browse files

Don't divide by the determinant in the gradient; add a default constructor

[[Imported from SVN: r8562]]
parent 7c46ed8e
No related branches found
No related tags found
No related merge requests found
...@@ -12,17 +12,17 @@ class OrthogonalMatrix ...@@ -12,17 +12,17 @@ class OrthogonalMatrix
{ {
public: public:
/** \brief Default constructor -- leaves the matrix uninitialized */
OrthogonalMatrix()
{}
/** \brief Constructor from a general matrix /** \brief Constructor from a general matrix
* *
* The input matrix gets normalized to det = 1. Since computing * It is not checked whether the matrix is really orthogonal
* the determinant is expensive you may not want to use this method
* if you know your input matrix to be orthogonal anyways.
*/ */
explicit OrthogonalMatrix(const Dune::FieldMatrix<T,N,N>& matrix) explicit OrthogonalMatrix(const Dune::FieldMatrix<T,N,N>& matrix)
: data_(matrix) : data_(matrix)
{ {}
data_ /= matrix.determinant();
}
/** \brief Const access to the matrix entries */ /** \brief Const access to the matrix entries */
const Dune::FieldMatrix<T,N,N>& data() const const Dune::FieldMatrix<T,N,N>& data() const
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment