benchopt / benchmark_tv_1d

TV Denoising in 1D
2 stars 7 forks source link

Fix ADMM test #48

Closed EnLAI111 closed 2 years ago

agramfort commented 2 years ago

Why do you need this @EnLAI111 ?

EnLAI111 commented 2 years ago

Why do you need this @EnLAI111 ?

I wrote a test of a function and its jacobian matrix in ADMM, and they are based on other functions imported by import_ctx.import_from() form other files. Without this, the test can't work.

tomMoral commented 2 years ago

Actually, I read too fast. You should not put this in the ADMM.py file but in the test file.