Use linesearch API in GradientDescent and ArmijoSGD
Open
enhancement
- Dominant language
- Python
- Stars
- 1.1k
- Forks
- 76
- Avg merge
- 2d 21h
- Merged PRs (30d)
- 1
Description
`GradientDescent` is currently just a subclass of `ProximalGradient`, which means that it only supports FISTA's linesearch.
`ArmijoSGD` also implements its own Armijo linesearch.
In both cases, it would be better to use our linesearch API. This would allow us to support more linesearch strategies and would reduce code duplication.
In that case, it's not clear if we need to keep the two classes or just keep one of the two.
@vroulet @Algue-Rythme
Contributor guide
Assessment
This issue has not been assessed yet.