search

GAA-UAM / scikit-fda

Functional Data Analysis Python package
https://fda.readthedocs.io
BSD 3-Clause "New" or "Revised" License
290 stars 51 forks source link

Remove L2Regularization #409

Closed vnmabus closed 2 years ago

vnmabus commented 2 years ago

L2Regularization is just TikhonovRegularization with the identity parameter. We should remove L2Regularization and add the second derivative as the default linear operator in TikhonovRegularization instead.

vnmabus commented 2 years ago

After internal discussion, we decided that the most sensible approach is to keep the name L2Regularization, which is more informative, and add optional linear_operator parameter (by default, the identity operator) to it. This has been done in 0bc020cecc1b83fa7821dd584977677db23ccd40.