clean up a bit

Author: dance858

CMakeLists.txt

@@ -1,6 +1,3 @@
 4. in the refactor, add consts
 10. For performance reasons, is it useful to have a dense matmul with A and B as dense matrices?
 11. right matmul, add broadcasting logic as in left matmul. Is this necessary?
-
-Going through all atoms to see that sparsity pattern is computed in initialization of hessian:
-3. trace - not ok

TODO.md

@@ -4,6 +4,7 @@
 #include "expr.h"
 #include "utils/CSR_Matrix.h"
 #include "utils/Timer.h"
+#include <stdbool.h>
 
 typedef struct
 {
@@ -13,7 +14,10 @@ typedef struct
     double time_eval_hessian;
     double time_forward_obj;
     double time_forward_constraints;
-} stats;
+
+    int nnz_affine;
+    int nnz_nonlinear;
+} Diff_engine_stats;
 
 typedef struct problem
 {
@@ -38,11 +42,13 @@ typedef struct problem
     bool jacobian_called;
 
     /* Statistics for performance measurement */
-    stats stats;
+    Diff_engine_stats stats;
+    bool verbose;
 } problem;
 
 /* Retains objective and constraints (shared ownership with caller) */
-problem *new_problem(expr *objective, expr **constraints, int n_constraints);
+problem *new_problem(expr *objective, expr **constraints, int n_constraints,
+                     bool verbose);
 void problem_init_derivatives(problem *prob);
 void free_problem(problem *prob);
 

include/problem.h

@@ -7,7 +7,8 @@ static PyObject *py_make_problem(PyObject *self, PyObject *args)
 {
     PyObject *obj_capsule;
     PyObject *constraints_list;
-    if (!PyArg_ParseTuple(args, "OO", &obj_capsule, &constraints_list))
+    int verbose = 1;
+    if (!PyArg_ParseTuple(args, "OO|p", &obj_capsule, &constraints_list, &verbose))
     {
         return NULL;
     }
@@ -49,7 +50,8 @@ static PyObject *py_make_problem(PyObject *self, PyObject *args)
         }
     }
 
-    problem *prob = new_problem(objective, constraints, (int) n_constraints);
+    problem *prob =
+        new_problem(objective, constraints, (int) n_constraints, verbose);
     free(constraints);
 
     if (!prob)