search

dask / dask-glm

BSD 3-Clause "New" or "Revised" License
75 stars 46 forks source link

Add default arguments to ADMM code #16

Closed mrocklin closed 7 years ago

mrocklin commented 7 years ago

I started trying to play with @hussainsultan 's ADMM code. However, it requires many parameters for which I do not know good values. Can we add sane defaults there? Do we know sane defaults?

Short term it would be useful to me to always be able to do the following for all solver routines:

result = func(X, y)
hussainsultan commented 7 years ago

Thanks for the call out. Thats sometime that i intended to clean in my initial PR but then got sidetracked by consolidation. I will take this on.

cicdw commented 7 years ago

This should be resolved in PR #22

hussainsultan commented 7 years ago

resolved by #22. closing