Update the move limit strategy.

MaxvdKolk commented 3 years ago

This updates the implementation of the move limit strategies:

- Adds a feasible region to intervening variables that is applied
separately within the construction (`build`) of the sub problem. This
simplifies the move limit implementation as no check needs to be
performed to enforce such feasible regions

- Changes the API to use the `__call__` rather than `update` to make it
more explicit that the lower/upper bounds are directly returns.

- Small changes in tests and examples to match the updated class names
and function calls.

Updates the move limit strategy from previous discussions:

Both sao/intervening_variables/mma.py and sao/intervening_variables/intervening.py add the alpha and beta properties to the intervening variables to indicate their feasible range. This still needs to be done for ConLin (and maybe the others too) but I was not completely sure on those values.
Pleas review changes in sao/move_limits/move_limit.py

artofscience commented 3 years ago

@aatmdelissen can you look at this. I recall you had some ideas on this yesterday.

artofscience commented 3 years ago

since we incroporated this in the newest "conceptual design", can we omit this PR? @MaxvdKolk

MaxvdKolk commented 3 years ago

Sure, lets just close and delete this branch.

artofscience / SAOR

Update the move limit strategy. #53