Fix broadcasting f-in-set in constraint macro #3632
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
x-ref #3629
So this line was never covered:
https://app.codecov.io/gh/jump-dev/JuMP.jl/commit/419c3344c9a62692e344da745765ac7daec95c01/blob/src/macros.jl#L839
The
Ref
was added here:https://github.com/jump-dev/JuMP.jl/pull/1438/files#diff-e628f2d30670b1af0cdd9c9c520aaddc44021cd503e169d767ec49c95f189264R288
I guess because we didn't yet have:
jump-dev/MathOptInterface.jl@ed08ee9
I think it's because only the
.∈
syntax supports this. You can't.
the in-fixin
operator: