Fix issue

Author: filikat

highs/ipm/hipo/ipm/Model.cpp

@@ -412,8 +412,6 @@ void Model::scale(OptionScaling opt) {
   // z -> C * z
   // where R is row scaling, C is col scaling.
 
-  if (!needScaling()) return;
-
   colscale_.resize(n_, 1.0);
   rowscale_.resize(m_, 1.0);
 
@@ -430,29 +428,20 @@ void Model::scale(OptionScaling opt) {
   applyScaling();
 }
 
-bool Model::needScaling() {
-  bool need_scaling = false;
+void Model::CRscaling() {
+  // Curtis-Reid scaling
 
-  // check entries of A
+  bool need_scaling = false;
   for (Int col = 0; col < n_; ++col) {
     for (Int el = A_.start_[col]; el < A_.start_[col + 1]; ++el) {
-      if (std::abs(A_.value_[el]) != 1.0) return true;
-    }
-  }
-
-  // check bounds
-  for (Int i = 0; i < n_; ++i) {
-    if (std::isfinite(lower_[i]) && std::isfinite(upper_[i])) {
-      const double diff = std::abs(lower_[i] - upper_[i]);
-      if (diff < kSmallBoundDiff || diff > kLargeBoundDiff) return true;
+      if (std::abs(A_.value_[el]) != 1.0) {
+        need_scaling = true;
+        break;
+      }
     }
   }
 
-  return false;
-}
-
-void Model::CRscaling() {
-  // Curtis-Reid scaling
+  if (!need_scaling) return;
 
   // Compute exponents for CR scaling of matrix A
   std::vector<Int> colexp(n_);

highs/ipm/hipo/ipm/Model.h

@@ -82,7 +82,6 @@ class Model {
   void computeNorms();
 
   void scale(OptionScaling opt);
-  bool needScaling();
   void applyScaling();
   void CRscaling();
   void onePassNormScaling();

highs/lp_data/HighsOptions.h

@@ -1282,7 +1282,7 @@ class HighsOptions : public HighsOptionsStruct {
         new OptionRecordString(kHipoScalingString,
                                "HiPO scaling option: \"curtis-reid\" "
                                "or \"norm\".",
-                               advanced, &hipo_scaling, kHipoCRscaling);
+                               advanced, &hipo_scaling, kHipoNormScaling);
     records.push_back(record_string);
 
     record_bool = new OptionRecordBool(