Skip to content

Conversation

odow
Copy link
Member

@odow odow commented Sep 26, 2022

Closes #2005

@odow odow added Type: Bug Submodule: Utilities About the Utilities submodule labels Sep 26, 2022
@odow odow merged commit 5673b92 into master Sep 26, 2022
@odow odow deleted the od/fix-unsupported-get branch September 26, 2022 03:52
@blegat
Copy link
Member

blegat commented Sep 26, 2022

Wasn't it already fixed by #2007 ?

@odow
Copy link
Member Author

odow commented Sep 26, 2022

That fixed part of the problem. But we needed this as well.

@blegat
Copy link
Member

blegat commented Sep 28, 2022

Getting a MethodError is the right behavior so I don't thing there is anything to fix. For non-variable constraints, this is also what you get. We could have a default fallback in src/attributes if we want to replace the MethodError by something else. Or we could say that all implementation should do it explicitly but then we need to update all of them in MOI.Utilities at least to have something consistent (and tests as well would be even better)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Submodule: Utilities About the Utilities submodule Type: Bug
Development

Successfully merging this pull request may close these issues.

MOIU._single_variable_flag
2 participants