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/
< zoq>
jeffin143[m]: nahh, glad I could help, I forgot that mlboard is header only as well.
< zoq>
rcurtin: I see, I thought each PR is failing for some strange reasons.
< rcurtin>
yeah, it's different for each one, it just seems like a lot more red Xs than green check marks :)
< walragatver[m]>
Hey jeffin143: birm: I am here
< jeffin143[m]>
Hi walragatver , here as well
< walragatver[m]>
jeffin143: I am currently reviewing #9
< jeffin143[m]>
walragatver: yes so finally I was able to build and run test on windows
< walragatver[m]>
I am happy that you were able to built on windows
< jeffin143[m]>
Yeah
favre49 has quit [Remote host closed the connection]
< walragatver[m]>
Okay so that's it for the discussion today
< walragatver[m]>
I will review all the PRs soon. I was slightly busy
< jeffin143[m]>
walragatver : no issues
< jeffin143[m]>
I will write test and rebase text pr
< jeffin143[m]>
And also come up with any test if possible for image
< jeffin143[m]>
Also planning to add support for logging audio summary
< jeffin143[m]>
Though it doesn't make any sense here
< jeffin143[m]>
Since we don't have any support for audio but I guess it ok to have
< walragatver[m]>
jeffin143: Okay if we don't need audio support what we can do is to leave it for now. When a use case for audio support we can implement it
< RyanBirminghamGi>
Hi jeffin143 and walragatver!
< jeffin143[m]>
walragatver: it's relatively easy to add so I don't have any trouble in supporting that
< jeffin143[m]>
I guess it would really take less time to add those
< jeffin143[m]>
Hi Ryan Birmingham (Gitter) : Thanks for the review :)
< RyanBirminghamGi>
Working with windows is frustrating, but I'm sure I don't have to tell you that now ;)
< walragatver[m]>
You can prioritize working on other supports like histograms it would be a good support
< walragatver[m]>
It looks there is connection issue at this moment
< walragatver[m]>
Okay connection is back again
< jeffin143[m]>
walragatver: ok then I will try to work on histogram next :)
< walragatver[m]>
Basically, you would be knowing that visualizing the objective is quite important to build good model
< jeffin143[m]>
Yes
< walragatver[m]>
Okay so that's it from my side.
< jeffin143[m]>
Yeah I also have nothing else
< walragatver[m]>
birm: Anything you would like to discuss something?
< RyanBirminghamGi>
This goes not just for this project, but everyone should remember to do their evaluations!
< walragatver[m]>
birm: Yes absolutely
< RyanBirminghamGi>
Other than that, I think we're ok. :)
< RyanBirminghamGi>
Ideally, pull requests would each be highly independent, but since this is a new repo, I understand that this does not always work. Just hoping we don't get any difficult conflicts. :)
< walragatver[m]>
birm: Yeah you are correct
< walragatver[m]>
jeffin143: birm: Have a nice week ahead. See you on Sunday