admm headers have a dependency issue

[smu160]

In general, headers should not be dependent upon other headers being included first. In admm.cpp, rearranging the order of those three headers causes compile time errors. Specifically, including admm.h (which is the desired order) prior to line_search.h and glmgen.h causes issues.

#include "line_search.h"
#include "glmgen.h"
#include "admm.h"

Attempted to fix by placing #include "glmgen.h" in admm.h, but this causes issues when building the trefide python package.

See:

trefide/src/admm.cpp

Line 8 in f87fdf1

#include "line_search.h"