06:45 < jenkins-mlpack2> Project docker ensmallen nightly build build #29: ABORTED in 2 hr 30 min: http://ci.mlpack.org/job/docker%20ensmallen%20nightly%20build/29/

07:07 < jenkins-mlpack2> Yippee, build fixed!

07:07 < jenkins-mlpack2> Project docker mlpack nightly build build #472: FIXED in 2 hr 53 min: http://ci.mlpack.org/job/docker%20mlpack%20nightly%20build/472/

13:09 < khizirsiddiqui> Hey rcurtin ! In PR 1836: https://github.com/mlpack/mlpack/pull/1836/ , just to be clear you suggest making evaluation metrics like accuracy, precision, recall, and then rmse and mae to be implemented in similar fashion as the metrics (lmetric.hpp). This will require changing already existing code, I am planning to change params in Evaluate()

13:09 < khizirsiddiqui> function to mostly 2; y_predicted, y_actual. This should work right?

13:25 < ssk99> rcurtin: I was looking at the move and copy assignment operators in rectangle_tree_impl that you added in #1934, I noticed that you haven't manipulated other.stat. Could you tell me if there was a reason for this?

13:32 < rcurtin> ssk99: I'm not sure why that is---other.stat should definitely be handled; so maybe that is a bug that should be fixed

13:33 < ssk99> Will do. :) Just wanted to confirm before I made a change

14:45 < walragatver> rcurtin: zoq: I think we should release mlpack without any furthur delay. I think we should release it in this weakend. This is just a suggestion considering the installation issue. If a PR will be remaining to be merged we will make a short release soon.

15:04 < rcurtin> walragatver: I think you're right. I'm not sure it's worth waiting on #1987 and #1493 anymore

15:05 < rcurtin> I'll do the release tonight unless those are immediately ready, and we can release 3.2.1 shortly afterwards

15:13 < zoq> sounds reasonable

16:12 < jeffin143> hacktoberfest in few days :)

16:12 < jeffin143> Time to get some free swags

16:36 < rcurtin> jeffin143: good point, I guess it is time for another free shirt :)

16:46 < zoq> jeffin143: Just got the registration email :)

16:59 < rcurtin> actually, I guess before I make the release, I can go ahead and merge the Julia bindings since they are approved... anyone opposed to me doing that?

17:00 < rcurtin> (I'm happy to wait if anyone wants to review it. it's not a huge deal either way)

17:00 < zoq> I think it's ready to get merged.

17:01 < jeffin> rcurtin , zoq , how about some Mlpack t shirt :)

17:01 < zoq> jeffin: Yeah, would be great to have, it's one the list :)

17:03 < jeffin143> Would love to acquire one :)

17:04 < zoq> Same :) maybe I have some time over the next days to do some research in that direction.

17:05 < zoq> I guess, the question is do we just go with the logo?

17:08 < jeffin143> Some , good c++ quotes along with it would do good

17:08 < jeffin143> May be I am not the best person to ask for creative ideas :-p

17:09 < zoq> Maybe we should ask on the mailinglist for ideas.

17:09 < jeffin143> I like the idea which rcurtin follows , in his mail the footer will have some funny quotes or Something , :)

17:10 < jeffin143> May be for t shirt we can't have many because , it's difficult to print with those many and always , so may be a common would be

17:10 < jeffin143> Anyways will be better if we ask in mailing list

17:45 < rcurtin> zoq: jeffin143: yeah, we've talked about an mlpack t-shirt in the past but never did it; I think it would be great

17:45 < rcurtin> personally I like the idea a simple shirt with just the logo, but that's just me :)

17:49 < rcurtin> actually, I thought about it, I won't merge the Julia bindings tonight because that won't leave enough time to see if there are any bugs that the CI checks didn't catch

17:50 < rcurtin> it seems too risky to merge and then immediately release :)