Skip to content

Commit

Permalink
Merge pull request ossimlabs#1 from JorgeGIlG/JorgeGIlG-patch-1
Browse files Browse the repository at this point in the history
RPC solver: Set system of equations only once
  • Loading branch information
JorgeGIlG authored Mar 1, 2022
2 parents 322b756 + 322bcab commit 4ce99c7
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/projection/ossimRpcSolver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -471,6 +471,9 @@ void ossimRpcSolver::solveCoefficients(NEWMAT::ColumnVector& coeff,
NEWMAT::ColumnVector tempCoeff;
NEWMAT::DiagonalMatrix weights((int)f.size());
NEWMAT::ColumnVector denominator(20);

// sets up the matrix to hold the system of equations
setupSystemOfEquations(m, r, x, y, z);

// initialize the weight matrix to the identity
//
Expand All @@ -493,9 +496,6 @@ void ossimRpcSolver::solveCoefficients(NEWMAT::ColumnVector& coeff,
}
#endif

// sets up the matrix to hold the system of equations
setupSystemOfEquations(m, r, x, y, z);

// solve the least squares solution. Note: the invert is used
// to do a Singular Value Decomposition for the inverse since the
// matrix is more than likely singular. Slower but more robust
Expand Down

0 comments on commit 4ce99c7

Please sign in to comment.