Skip to content

fix highspy version check for keyboard interrupt#3511

Merged
mrmundt merged 1 commit intoPyomo:mainfrom
michaelbynum:highspy_version
Mar 10, 2025
Merged

fix highspy version check for keyboard interrupt#3511
mrmundt merged 1 commit intoPyomo:mainfrom
michaelbynum:highspy_version

Conversation

@michaelbynum
Copy link
Copy Markdown
Contributor

Summary/Motivation:

The logic used to check the version of HiGHS before using HandleKeyboardInterrupt in #3509 was not quite correct.

Changes proposed in this PR:

  • Ensure the version is greater than (1,8).

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.

@michaelbynum michaelbynum requested a review from mrmundt March 10, 2025 12:58
Copy link
Copy Markdown
Member

@jsiirola jsiirola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@mrmundt mrmundt merged commit a1cf555 into Pyomo:main Mar 10, 2025
34 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants