Skip to content
This repository has been archived by the owner on Dec 6, 2023. It is now read-only.

CDClassifer consistency #110

Open
arthurmensch opened this issue Feb 20, 2017 · 1 comment
Open

CDClassifer consistency #110

arthurmensch opened this issue Feb 20, 2017 · 1 comment

Comments

@arthurmensch
Copy link

The API of CDClassifier and SAG(A)Classifier should be uniformized.

For the moment, the objective of CDClassifier is parametrized by C, loss alpha and penalty,
whereas SAG(A)Classifier sets C = 1 / n_samples, uses beta instead of alpha and a specific alpha to add a ridge regularization. Wouldn't it be better to align the CDClassifier API on the SAG(A)Classifier ?

@fabianp
Copy link
Member

fabianp commented Feb 20, 2017

+1 from me.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants