-
Notifications
You must be signed in to change notification settings - Fork 56
Closed
Labels
Description
In the add_client_assign_integer_variable factory, the variable lower and upper bounds are hard-coded to 0 and 1, respectively. This is correct for the
Moreover, in the CLSCP-SO the client assignment variables are not constrained to be integers, which we already account for by setting them as continuous.
So we need to either:
- rename the method and add keywords for lower/upper bounds
add_client_assign_integer_variable->add_client_assign_variable
- create a new method
add_client_assign_continuous_variable
Choice (1.) is the most elegant and maintainable solution, IMHO.