You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openoffice.apache.org by pf...@apache.org on 2015/09/24 04:36:24 UTC

svn commit: r1704975 - /openoffice/trunk/main/sccomp/source/solver/solver.cxx

Author: pfg
Date: Thu Sep 24 02:36:24 2015
New Revision: 1704975

URL: http://svn.apache.org/viewvc?rev=1704975&view=rev
Log:
i124091 - Drop check for nonlinearity

It appears the check for non-linearity is causing failures for cases that 
can be solved by CoinMP and other solvers. We don't really have a 
non-linear solver but it doesn't make sense to check for linearity here.

Thanks to Ted Ralphs for pointing the exact code causing the failure.

Modified:
    openoffice/trunk/main/sccomp/source/solver/solver.cxx

Modified: openoffice/trunk/main/sccomp/source/solver/solver.cxx
URL: http://svn.apache.org/viewvc/openoffice/trunk/main/sccomp/source/solver/solver.cxx?rev=1704975&r1=1704974&r2=1704975&view=diff
==============================================================================
--- openoffice/trunk/main/sccomp/source/solver/solver.cxx (original)
+++ openoffice/trunk/main/sccomp/source/solver/solver.cxx Thu Sep 24 02:36:24 2015
@@ -369,12 +369,6 @@ void SAL_CALL SolverComponent::solve() t
             double fInitial = aCellsIter->second.front();
             double fCoeff   = aCellsIter->second.back();       // last appended: coefficient for this variable
             double fTwo     = lcl_GetValue( mxDoc, aCellsIter->first );
-
-            bool bLinear = rtl::math::approxEqual( fTwo, fInitial + 2.0 * fCoeff ) ||
-                           rtl::math::approxEqual( fInitial, fTwo - 2.0 * fCoeff );
-            // second comparison is needed in case fTwo is zero
-            if ( !bLinear )
-                maStatus = lcl_GetResourceString( RID_ERROR_NONLINEAR );
         }
 
         lcl_SetValue( mxDoc, *aVarIter, 0.0 );      // set back to zero for examining next variable