Agreed. I have made the change in the sdrsac-cpp repo and the error seen previously is no longer observed. Now I need to verify correct execution. Thanks, Ryan!
On Mon, Sep 2, 2019 at 1:38 PM Ryan Curtin <[email protected]> wrote: > On Mon, Sep 02, 2019 at 01:35:18PM -0700, Joe Dinius wrote: > > Ryan, > > > > Thanks so much for your awesome explanation and for looking into it! > It's > > really amusing (to me anyways) that, just before reading this, I was > going > > through the armadillo docs and read the following: > > > > - *Can I use the C++11 auto keyword with Armadillo objects and/or > > expressions?* > > Use of C++11 *auto* is *not recommended* with Armadillo objects and > > expressions. Armadillo has a template meta-programming framework which > > creates lots of short lived temporaries that are not handled by > *auto*. > > > > and *immediately* thought "maybe this is what's happening with my SDP > > code". HAHAHA. I will try it out and let you know how it goes. > > Ha! Yeah, hopefully it works. But even so like I said it's a little > strange for auto to fail in this way, since the type of n_rows should be > easily deducible. > > -- > Ryan Curtin | "What? Facts?" > [email protected] | - Joe Cairo >
_______________________________________________ mlpack mailing list [email protected] http://knife.lugatgt.org/cgi-bin/mailman/listinfo/mlpack
