Update HiGHS to v1.10.0

[nickodell]

Update HiGHS to v1.10.0 in order to fix scipy/scipy#22188.

[rgommers]

Thanks! I assume this is draft because you want a SciPy PR with green CI before merging this?

[nickodell]

Thanks! I assume this is draft because you want a SciPy PR with green CI before merging this?

It's a draft because there are a few things I am not sure how to fix. Right now I get some compiler warnings from Highs:

[1242/1450] Compiling C++ object scipy/optimize/_highspy/_...u.so.p/.._.._.._subprojects_highs_src_highs_bindings.cpp.o
In file included from ../subprojects/highs/src/lp_data/HighsAnalysis.h:17,
                 from ../subprojects/highs/src/util/HFactor.h:21,
                 from ../subprojects/highs/src/simplex/HighsSimplexAnalysis.h:22,
                 from ../subprojects/highs/src/simplex/HSimplexNla.h:17,
                 from ../subprojects/highs/src/simplex/HEkk.h:15,
                 from ../subprojects/highs/src/lp_data/HighsLpSolverObject.h:16,
                 from ../subprojects/highs/src/lp_data/HighsRanging.h:16,
                 from ../subprojects/highs/src/Highs.h:19,
                 from ../subprojects/highs/src/highs_bindings.cpp:9:
../subprojects/highs/src/util/HighsTimer.h: In constructor 'HighsTimer::HighsTimer()':
../subprojects/highs/src/util/HighsTimer.h:46:14: warning: unused variable 'i_clock' [-Wunused-variable]
   46 |     HighsInt i_clock = clock_def("Run HiGHS");
      |              ^~~~~~~
../subprojects/highs/src/util/HighsTimer.h: In member function 'void HighsTimer::resetHighsTimer()':
../subprojects/highs/src/util/HighsTimer.h:103:14: warning: unused variable 'i_clock' [-Wunused-variable]
  103 |     HighsInt i_clock = clock_def("Run HiGHS");
      |              ^~~~~~~
../subprojects/highs/src/highs_bindings.cpp: In function 'std::tuple<HighsStatus, pybind11::array_t<double, 17>, int, pybind11::array_t<int, 17> > highs_getReducedRowSparse(Highs*, HighsInt)':
../subprojects/highs/src/highs_bindings.cpp:335:12: warning: unused variable 'num_col' [-Wunused-variable]
  335 |   HighsInt num_col = h->getNumCol();
      |            ^~~~~~~

I don't quite understand why these warnings appear, because our meson configuration ought to be ignoring them.

[rgommers]

I don't quite understand why these warnings appear, because our meson configuration ought to be ignoring them.

Compile flags like -Wno-unused-variable are per target, and the highspy/_core extension module doesn't have any warning suppression flags. You can see this in the meson.build file, and also in the generated build/build.ninja (just search for highs_bindings.cpp there).

To fix it, we need an extra commit here. Ideally it's fixed upstream and then that fix included here. It's fine to do it out of order and fix it first here though, if that's easier.