Skip to content
Snippets Groups Projects
Commit 9a1b9ee4 authored by Elias Pipping's avatar Elias Pipping
Browse files

Merge branch 'feature/box-constraints-as-intervals' into 'master'

Expose box constraints as intervals

As the changes to obstacletnnmgstep.hh illustrate, this allows us to replace code like

```c++
{
    if (coarseCorrection_[i][j] < defectConstraint.lower(j))
        coarseCorrection_[i][j] = defectConstraint.lower(j);
    if (coarseCorrection_[i][j] > defectConstraint.upper(j))
        coarseCorrection_[i][j] = defectConstraint.upper(j);
}
```
with
```c++
    coarseCorrection_[i][j]
        = defectConstraint[j].projectIn(coarseCorrection_[i][j]);
```
which is no less efficient yes far easier to read.

I have no removed any functionality. The set of features that a `BoxConstraint` currently comes with is quite odd, though. It ha an `operator-=` but no `operator+=` e.g.

See merge request !6
parents dc6e79d8 cd1bed32
No related branches found
No related tags found
1 merge request!6Expose box constraints as intervals
Pipeline #
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment