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

Tidy names of sets #266

Merged
merged 1 commit into from
Nov 15, 2023
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 15 additions & 15 deletions src/MOI_wrapper.jl
Original file line number Diff line number Diff line change
Expand Up @@ -455,20 +455,20 @@
function MOI.add_constraint(
model::Optimizer,
x::MOI.VariableIndex,
lt::MOI.LessThan{Float64},
set::MOI.LessThan{Float64},
)
_throw_if_solved(model, x, lt)
_throw_if_solved(model, x, set)
MOI.throw_if_not_valid(model, x)
if isnan(lt.upper)
error("Invalid upper bound value $(lt.upper).")
if isnan(set.upper)
error("Invalid upper bound value $(set.upper).")

Check warning on line 463 in src/MOI_wrapper.jl

View check run for this annotation

Codecov / codecov/patch

src/MOI_wrapper.jl#L463

Added line #L463 was not covered by tests
end
if _has_upper_bound(model, x)
error("Upper bound on variable $x already exists.")
end
if _is_fixed(model, x)
error("Variable $x is fixed. Cannot also set upper bound.")
end
ub = _check_value(lt.upper)
ub = _check_value(set.upper)
model.variable_info[x.value].has_upper_bound = true
KN_set_var_upbnd(model.inner, _c_column(x), ub)
ci = MOI.ConstraintIndex{MOI.VariableIndex,MOI.LessThan{Float64}}(x.value)
Expand All @@ -490,20 +490,20 @@
function MOI.add_constraint(
model::Optimizer,
x::MOI.VariableIndex,
gt::MOI.GreaterThan{Float64},
set::MOI.GreaterThan{Float64},
)
_throw_if_solved(model, x, gt)
_throw_if_solved(model, x, set)
MOI.throw_if_not_valid(model, x)
if isnan(gt.lower)
error("Invalid lower bound value $(gt.lower).")
if isnan(set.lower)
error("Invalid lower bound value $(set.lower).")

Check warning on line 498 in src/MOI_wrapper.jl

View check run for this annotation

Codecov / codecov/patch

src/MOI_wrapper.jl#L498

Added line #L498 was not covered by tests
end
if _has_lower_bound(model, x)
error("Lower bound on variable $x already exists.")
end
if _is_fixed(model, x)
error("Variable $x is fixed. Cannot also set lower bound.")
end
lb = _check_value(gt.lower)
lb = _check_value(set.lower)
model.variable_info[x.value].has_lower_bound = true
KN_set_var_lobnd(model.inner, _c_column(x), lb)
ci = MOI.ConstraintIndex{MOI.VariableIndex,MOI.GreaterThan{Float64}}(x.value)
Expand Down Expand Up @@ -564,12 +564,12 @@
function MOI.add_constraint(
model::Optimizer,
x::MOI.VariableIndex,
eq::MOI.EqualTo{Float64},
set::MOI.EqualTo{Float64},
)
_throw_if_solved(model, x, eq)
_throw_if_solved(model, x, set)
MOI.throw_if_not_valid(model, x)
if isnan(eq.value)
error("Invalid fixed value $(eq.value).")
if isnan(set.value)
error("Invalid fixed value $(set.value).")

Check warning on line 572 in src/MOI_wrapper.jl

View check run for this annotation

Codecov / codecov/patch

src/MOI_wrapper.jl#L572

Added line #L572 was not covered by tests
end
if _has_lower_bound(model, x)
error("Variable $x has a lower bound. Cannot be fixed.")
Expand All @@ -580,7 +580,7 @@
if _is_fixed(model, x)
error("Variable $x is already fixed.")
end
eqv = _check_value(eq.value)
eqv = _check_value(set.value)
model.variable_info[x.value].is_fixed = true
KN_set_var_fxbnd(model.inner, _c_column(x), eqv)
ci = MOI.ConstraintIndex{MOI.VariableIndex,MOI.EqualTo{Float64}}(x.value)
Expand Down
Loading