Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add JuMP error message when non-optimal (typ infeasible) #417

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Bill-Becker
Copy link
Collaborator

Currently only add first entry of the message which is typically the most informative and human readable.

Currently only add first entry of the message which is typically the most informative and human readable.
@Bill-Becker
Copy link
Collaborator Author

@rathod-b can you check this out?

@rathod-b
Copy link
Collaborator

rathod-b commented Jul 2, 2024

@rathod-b can you check this out?

Hi Bill, I did a quick test and wanted to better understand the intent of this change. Are we trying to return a dict when models are infeasible, irrespective of what information is contained in the dictionary? I did a simple test and got a dictionary in response instead of the model object, which is an improvement. But the error key contained a very long error (see below) which doesnt make it clear that the problem was infeasible.

image

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.

2 participants