Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
    Matthew Rocklin
    @mrocklin
    Hi @all I've made this chat room for dask-glm conversation
    With dask/dask-glm#21 I've now passed through all of @moody-marlin 's original routines and made them as fast as they're likely to get with the current systems. We have some super-basic testing in place. What's next? Some questions:
    • In https://github.com/dask/dask-glm/issues/5#issuecomment-27541381 @mcg1969 said that he was waiting on a notebook from @moody-marlin in order to explore slow stepsize evaluation. Is this still waiting on something?
    • What is the best way to use @jcrist here?
    • How do we start comparing against SAS or H2O?
    • I have yet to look at the ADMM code. Is this in a ready state?
    Chris White
    @cicdw
    I made a dask/dask-glm#22 that puts the ADMM code in a ready state for anyone to review
    that was supposed to say “pull request"
    Also, @mcg1969 — you are correct that the line searches actually aren’t requiring as many function evaluations as I believed, so I don’t have anything to send you — however, the BFGS code is failing a high-level test (sum of predictions = sum of actuals) and I can’t seem to determine why, so any insights there would be helpful
    Hussain Sultan
    @hussainsultan
    here!
    beautiful
    Matthew Rocklin
    @mrocklin
    Heh, I hadn't seen that
    Matthew Rocklin
    @mrocklin
    @moody-marlin is there a reason why you alias lf = curr_val in compute_stepsize and never update it? It seemed like it might be the sort of thing that you would update lf = func from time to time.
    Chris White
    @cicdw
    @mrocklin sorry so late to this; i need a desktop app for gitter to send me notifications! lf is the base function value that we are comparing against for "sufficient decrease", so once the new func value is "sufficiently" smaller than lf we exit -- however, there was no reason why i chose to redefine curr_val to lf