ChanServ changed the topic of #mlpack to: "mlpack: a fast, flexible machine learning library :: We don't always respond instantly, but we will respond; please be patient :: Logs at http://www.mlpack.org/irc/
< AakashkaushikGit>
Hey @shrit i am working to see the difference by removing the `boost::visitors`
< AakashkaushikGit>
and instead using virtual inheritance, I was running a bit busy due to exams but i will start working on it.
seeni_ has joined #mlpack
seeni_ has quit [Quit: seeni_]
ImQ009 has joined #mlpack
kyrre has joined #mlpack
< AakashkaushikGit>
Hey `BOOST_REQUIRE(x)` should be translated into something like `REQUIRE(x == true)` ?
< AnushKiniGitter[>
Yes @Aakash-kaushik. You can also do '''
< AnushKiniGitter[>
(edited) ... do ''' => ... do
< AnushKiniGitter[>
(edited) ... @Aakash-kaushik. You can also do => ... @Aakash-kaushik. ```REQUIRE(x)``` should also work.
< AakashkaushikGit>
@Abilityguy thanks, this seems like a more elegant solution.
kyrre has quit [Quit: Connection closed for inactivity]
< jeffin143[m]>
shrit : planning to migrate serialisation test to catch?
< jeffin143[m]>
Are you ok with it ??
< jeffin143[m]>
There would be some merge conflicts if I am not wrong ?? Would you be ok to handle those ??
< AakashkaushikGit>
Hey @jeffin143 i can handle most of the remaining tests so if you are actively working on one that doesn't have a PR, let me know so i don't take up the same
< shrit[m]>
@jeffin143 I am not planning to migrate anything at this time, but I prefere that you wait until cereal is merged, it is very close to be merged at this time. You can do a pull request and do a migration of course, but we can wait for merging I think.