Omar Shrit

@arthiondaena okay for sure, I will give it a review in the next couple of days

I have no idea how this went to stale :smile:

@rcurtin not yet, probably there is a need for one or two PR's and then I need to do some cleaning.

I will move the part related to `MakeAlias` implementation for armadillo to a new PR, otherwise, it would not make sense to add the `offset` in the middle of the...

I am closing this one since most of the modifications have alread been integrated as a part of different PR's

@SIDHARTH-7, this starts to look good, please add a couple of tests so we can review it.

@SIDHARTH-7 Good, idea it would be better to rewrite these tests indeed :+1:

@SIDHARTH-7 please have a look at the errors, they are fairly simple to understand, you need to adapt the Separable Conv layer code to make it look like mlpack 4.0,...

@SIDHARTH-7 could you start testing this PR against the model repository and see if the Mobilenet model compiles?

Also, could you resolve the conflicted files?