Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Correctly handling the case λmax = 0. #53

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Correctly handling the case λmax = 0. #53

wants to merge 4 commits into from

Commits on Sep 6, 2020

  1. Correctly handling the case λmax = 0. 

    Two changes:
1) Change to computeλ to ensure λmax = 0 leads to an output of [0] and 
not [NaN, ..., NaN].
2) Change to fit! to ensure the case where autoλ = true and λmax = 0 is 
handled correctly (rather than throwing an error).
    @barankarakus
    barankarakus committed Sep 6, 2020
    Configuration menu
    Copy the full SHA
    3ccd5f0 View commit details
    Browse the repository at this point in the history

Commits on Sep 20, 2020

  1. Update src/Lasso.jl 

    Changing spelling of 'regularisation'.
    @barankarakus
    barankarakus authored Sep 20, 2020
    Configuration menu
    Copy the full SHA
    f4a3923 View commit details
    Browse the repository at this point in the history

  2. Replacing equality with approximate equality.

    Baran Karakus committed Sep 20, 2020
    Configuration menu
    Copy the full SHA
    5636fed View commit details
    Browse the repository at this point in the history

  3. Added test for case: zero variation in y.

    Baran Karakus committed Sep 20, 2020
    Configuration menu
    Copy the full SHA
    5ac04f7 View commit details
    Browse the repository at this point in the history