Ryan Curtin

Results 312 comments of Ryan Curtin

@shrit actually, the problem would still exist; we'd simply need to implement it on Azure instead of AppVeyor. :+1:

Just checking in on this one; @shrit do you want to finish it? I'm not sure if the current approach is the right one. It looks like the implemented copy...

I'm not familiar with the SAC code at all, but my guess is that there is probably an Armadillo alias somewhere in here, and when we have the copy constructor...

@shrit what do you think about this one? I'm not sure it's still necessary after the ann-vtable refactoring, but you are more familiar with it than I am. :+1:

I spent a long time trying to find additional places to add OpenMP to accelerate `mnist_cnn` further, but you already got most of everything. :) Here's a patch that adds...

Thanks for this awesome improvement @shubham1206agra!

Hi @heisenbuug, the `PARAM_FLAG()` parameter is meant to define just one parameter. You would need to add another one for `no_sanity_checks` and you would also need to add appropriate documentation...

Python bindings are done with #2787. I added a 'status' section to the top in case anyone wants to try any of the other languages. It should be easier now...

Hi @Anmol2001, I haven't seen any PRs for other languages yet, so please feel free to go ahead. :+1: Please use the `CheckInputMatrices()` functions that @NippunSharma wrote. :)

@Anmol2001 looks like you're running out of RAM... maybe use fewer cores when compiling?