Fix: Get SCIP solving time considering float number with some text

[henriquejsfj]

This fix does not change previous behavior and handles the case of a string with a float number plus some text.
Fix #3233

Legal Acknowledgement

By contributing to this software project, I have read the contribution guide and agree to the following terms and conditions for my contribution:

1. I agree my contributions are submitted under the BSD license.
2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.40%. Comparing base (8561f48) to head (3ce7416).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3234   +/-   ##
=======================================
  Coverage   88.39%   88.40%           
=======================================
  Files         848      848           
  Lines       95322    95322           
=======================================
+ Hits        84259    84267    +8     
+ Misses      11063    11055    -8     

Flag	Coverage Δ
linux	86.34% <ø> (ø)
osx	76.19% <ø> (ø)
other	86.54% <ø> (+<0.01%)	⬆️
win	83.84% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mrmundt]

@henriquejsfj - Thanks for this PR! We think this looks fine. We test against SCIP 8.0 on our Jenkins, not 9.0, so we don't have a test to verify this behavior. Is it possible for you to add a small test for this edge case?

Unrelated: can you please edit the PR description to add back in the template? The legal acknowledgement is a requirement.

As for the failing test, that is unrelated. I have a different PR up to fix that.

[henriquejsfj]

@mrmundt Thanks for your reply.

I tested an example from docs and it was working, so I investigated and found the edge case:

solver = SolverFactory("scip")
solver.options['reoptimization/enable'] = True

By adding this option both SCIP 8 and 9 will return that bigger string breaking the code.

Previously, when I used SCIP 7 with Pyomo 6.3 I had no issues.

[mrmundt]

@henriquejsfj - I stand corrected! We do have SCIP 9.0 available on one test suite in GHA - linux/3.11/conda. Could you please add a test case to the test_scip.py file? (It can be a bit hard to find: https://github.com/Pyomo/pyomo/blob/main/pyomo/solvers/tests/mip/test_scip.py )

[henriquejsfj]

@mrmundt - Sure, just added it. It is always a good idea to add a test case to a bug that was not caught before!