Skip to content

Commit cde8bc0

Browse files
Transurgeonclaude
andcommitted
Fix lint and remove section comment banners
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 97b08bb commit cde8bc0

3 files changed

Lines changed: 0 additions & 17 deletions

File tree

cvxpy/reductions/solvers/nlp_solvers/diff_engine/converters.py

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -20,24 +20,17 @@
2020

2121
import cvxpy as cp
2222
from cvxpy.expressions.constants.parameter import Parameter
23-
2423
from cvxpy.reductions.solvers.nlp_solvers.diff_engine.helpers import (
2524
_make_dense_left_matmul,
2625
_make_dense_right_matmul,
2726
_make_sparse_left_matmul,
2827
_make_sparse_right_matmul,
2928
_to_dense_float,
30-
build_param_dict,
31-
build_var_dict,
3229
normalize_shape,
3330
)
3431
from cvxpy.reductions.solvers.nlp_solvers.diff_engine.registry import ATOM_CONVERTERS
3532

3633

37-
# ---------------------------------------------------------------------------
38-
# Matmul / multiply converters (need param_dict for parameter support)
39-
# ---------------------------------------------------------------------------
40-
4134
def _convert_matmul(expr, children, var_dict, n_vars, param_dict):
4235
"""Convert matrix multiplication A @ f(x), f(x) @ A, or X @ Y."""
4336
left_arg, right_arg = expr.args
@@ -106,9 +99,6 @@ def _convert_multiply(expr, children, var_dict, n_vars, param_dict):
10699
return _diffengine.make_multiply(children[0], children[1])
107100

108101

109-
# ---------------------------------------------------------------------------
110-
# Main conversion entry point
111-
# ---------------------------------------------------------------------------
112102

113103
def convert_expr(expr, var_dict, n_vars, param_dict=None):
114104
"""Convert a CVXPY expression to a C diff engine expression.

cvxpy/reductions/solvers/nlp_solvers/diff_engine/helpers.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,6 @@ def _chain_add(children):
4141
return result
4242

4343

44-
# ---------------------------------------------------------------------------
45-
# Matmul helpers (handle param_node insertion for SparseDiffPy API)
46-
# ---------------------------------------------------------------------------
4744

4845
def _make_sparse_left_matmul(param_node, child, A):
4946
if not isinstance(A, sparse.csr_matrix):
@@ -79,9 +76,6 @@ def _make_dense_right_matmul(param_node, child, A):
7976
param_node, child, A.flatten(order='C'), m, n)
8077

8178

82-
# ---------------------------------------------------------------------------
83-
# Variable / parameter dict builders
84-
# ---------------------------------------------------------------------------
8579

8680
def build_var_dict(inverse_data):
8781
"""Build {var_id: C variable capsule} mapping from InverseData."""

cvxpy/reductions/solvers/nlp_solvers/diff_engine/registry.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
from sparsediffpy import _sparsediffengine as _diffengine
2525

2626
import cvxpy as cp
27-
2827
from cvxpy.reductions.solvers.nlp_solvers.diff_engine.helpers import (
2928
_chain_add,
3029
normalize_shape,

0 commit comments

Comments
 (0)