Skip to content

Add InfiniteOpt extension#17

Open
DimitriAlston wants to merge 3 commits into
PSORLab:mainfrom
DimitriAlston:infiniteopt
Open

Add InfiniteOpt extension#17
DimitriAlston wants to merge 3 commits into
PSORLab:mainfrom
DimitriAlston:infiniteopt

Conversation

@DimitriAlston

Copy link
Copy Markdown
Member

No description provided.

@DimitriAlston

Copy link
Copy Markdown
Member Author

@pulsipher, do you have any thoughts on this?

@pulsipher

pulsipher commented Jun 24, 2026

Copy link
Copy Markdown

Thanks for taking the initiative on this. I will take a closer look later this week when I have some more time. An initial concern I have that using all_parameters and all_variables could easily lead to silent modeling errors. Is there a reason to not have the user more explicitly indicate what variables/parameters should be used?

A good example would be the API used by MathOptAI.

@DimitriAlston

DimitriAlston commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

I think all_variables is fine, even if every variable is not an input to each function. I agree all_parameters should be changed to take the specific infinite parameter as a function input so that we aren't using the wrong one incorrectly, if/when there are multiple. We could also make the user inputs more explicit, but I'm not sure if it is necessary when they define the appropriate variables in advance.

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