04:11 < AakashkaushikGit> I think doing just `make` on colab is fine because collab gives you plenty of ram so don't have to actually find out thr number of CPU cores assigned to you. the only reason for selecting number of core with the `j` flag is so that the more number of cores the more things can be processed parallely and that fills up the ram pretty quickly and terminates the build process on systems where ram is a bottleneck. And i

04:11 < AakashkaushikGit> don't think that's a problem with colab.

04:53 < AakashkaushikGit> btw @zoq making the members of the `layer.hpp` base class private, i don't think that makes sense because the derived classes will still have to define things like deterministic, weights and all that even after them having in the base class, can we go ahead with protected instead of private ?

04:54 < AakashkaushikGit> Also if anyone else is online please let me know what you think about this ?

08:03 < kaushal07wickGit> I am working on src/mlpack/methods/ann/layer/celu.hpp and I came across this template<typename DataType>

08:03 < kaushal07wickGit> void Backward(const DataType& input, const DataType& gy, DataType& g);

08:04 < kaushal07wickGit> do i have to change this this into specific InputType and OutputType types

08:09 < kaushal07wickGit> @zoq any additional changes or instructions for this .

08:59 < AakashkaushikGit> Hi @kaushal07wick are you working from the mlpack/master branch ?

12:10 < zoq> AakashkaushikGit: The derived class does inherit a full parent object, which contains any private members which that class declares. So I'm not sure I can see the issue, maybe I missed something?

12:11 < zoq> I updated the Dropout class - https://github.com/mlpack/mlpack/pull/2777/files#diff-d72b13bb3314316cbc484e65d1215e5f11cc7506af05ac87c4c3427dd45f925a and it worked just fine.

12:25 < zoq[m]> Yes, the easiest would be to take a look at an exsisting env use and implement the same interface.

14:42 < rcurtin[m]> A Boodhayana S Vishwamithra: be sure to take a look at https://www.mlpack.org/community.html; we've written up a guide for people who are interested in getting involved there

14:56 < AakashkaushikGit> Hey @zoq i think i missed that part, i didn't saw that we had a whole base class object in those derived classes. I think i will have to check again it probably is me who is missing something.

14:56 < AakashkaushikGit> It is*

17:27 < kaushal07wickGit> hey @Aakash-kaushik I made different feature branch

17:28 < kaushal07wickGit> from the mlpack/master branch

18:03 < rcurtin[m]> g

18:03 < rcurtin[m]> oops, wrong window! :)