search

cvxgrp / diffcp

Differentiation through cone programs
Apache License 2.0
91 stars 19 forks source link

Initial C++ projection #40

Open bamos opened 4 years ago

bamos commented 4 years ago

Smaller PR of just this from #39. Will send in the refinement PR on top of this once it's merged

I've replaced pi with the C++ version, but the tests are currently calling into the Python code for the individual projections and then calling into C++ for the overall test of pi. If it sounds good to you all I can remove the Python projection code and then make the individual projection tests check the C++ code.

sbarratt commented 3 years ago

LGTM on an initial pass. Do you have any benchmarks for this?