06:13 < akhandait> zoq: In the Sequential object, the destructor only deletes the layers in the Sequential object if `model == false`. Can I know the reason why it is this way? When we use the Sequenial object with `model == true`, the memory of the layers within gets leaked.

06:15 < akhandait> zoq: Should we add a public function to the Sequential object which allows us to manually delete all the layers as the destructor won't do it. This way, we won't have to change the recurrent layer implementation.

12:06 < jenkins-mlpack2> Project docker mlpack nightly build build #149: STILL UNSTABLE in 6 hr 52 min: http://ci.mlpack.org/job/docker%20mlpack%20nightly%20build/149/

14:26 < jenkins-mlpack2> Project mlpack - git commit test build #64: STILL UNSTABLE in 33 min: http://ci.mlpack.org/job/mlpack%20-%20git%20commit%20test/64/

14:26 < jenkins-mlpack2> noreply: Merge pull request #1583 from rcurtin/cmake-fixes

14:59 < travis-ci> mlpack/mlpack#5657 (master - b5d3720 : Ryan Curtin): The build has errored.

14:59 < travis-ci> Change view : https://github.com/mlpack/mlpack/compare/d963b4b54162...b5d3720fba41

14:59 < travis-ci> Build details : https://travis-ci.org/mlpack/mlpack/builds/465998353

16:46 < jenkins-mlpack2> Project mlpack - git commit test build #65: STILL UNSTABLE in 33 min: http://ci.mlpack.org/job/mlpack%20-%20git%20commit%20test/65/

16:46 < jenkins-mlpack2> noreply: Merge pull request #1591 from rcurtin/travis-keyserver

17:07 < zoq> rcurtin: Do you think if we restart jenkins, anonymous users can see the build status again? the option is enabled.

17:13 < zoq> akhandait: It's because one layer could be hold by multiple layer. So there is not a single deconstruction chain.

17:15 < zoq> You could set the model parameter and call 'delete object' to delete the layer.

17:15 < zoq> Isn't that the same, maybe I missed something.

17:16 < travis-ci> mlpack/mlpack#5658 (master - 7e05c30 : Ryan Curtin): The build passed.

17:16 < travis-ci> Change view : https://github.com/mlpack/mlpack/compare/b5d3720fba41...7e05c306405a

17:16 < travis-ci> Build details : https://travis-ci.org/mlpack/mlpack/builds/466066301

17:32 < akhandait> zoq: To be clear, are you saying that we can set `model = true` and then delete the layers that we added to the sequential object individually?

17:36 < rcurtin> zoq: let's try it and see. thanks for looking into it :)

17:39 < rcurtin> zoq: looks like no success, I'm still not able to look at jobs anonymously

17:39 < rcurtin> the plane internet is too bad for me to try and debug further

17:46 < zoq> akhandait: Right, I agree this is not the best option.

17:48 < akhandait> zoq: Yeah, we can't do that when we don't maintain pointers to those layers and directly add them to the Sequential object like this

17:48 < akhandait> Sequential<> a = new Sequential<>();

17:48 < akhandait> a->Add<Linear<> >(a, b);

17:48 < zoq> right

17:49 < akhandait> Now we have no way to free the memory of that Linear layer

17:50 < akhandait> I think we can call the delete visitor on the Sequential layer for that, but I am not sure

17:51 < zoq> have to think about that, but if you like to test it out with a couple of examples, please feel free :)

17:52 < akhandait> Okay, I will try and get back.

17:53 < zoq> Sounds good to me.

17:56 < akhandait> zoq: Do you think the Transposed conv PR is ready to be merged? #1493

17:57 < zoq> akhandait: Will take a look at the PR, later today.

17:57 < akhandait> zoq: Sure, whenever you are free

19:44 < ShikharJ_> akhandait: I would appreciate if we could wait a bit on that. Though I'm confident on most of the aspects of the PR, there are some things that I wish to test out, before we merge that if it's okay with you?