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/
jeffin143 has quit [Read error: Connection reset by peer]
ssk99 has joined #mlpack
< ssk99> Hi guys, how do I make cmake use python3.6 instead of python2.7 while compiling from source? Thanks
< ssk99> Ah I figured it out. Just have to use the -DPYTHON_EXECUTABLE:FILEPATH=/path/to/python_version
< ssk99> Just a suggestion but I think this should be included among the flags listed in http://www.mlpack.org/doc/stable/doxygen/build.html#build_config
ssk99 has quit [Remote host closed the connection]
jeffin143 has joined #mlpack
< jeffin143> @zoq : you there ?
< jeffin143> For loading image , can't I give arma::mat as output type to load an image ?
xiaohong has joined #mlpack
jeffin143 has quit [Ping timeout: 245 seconds]
xiaohong has quit [Ping timeout: 276 seconds]
< jenkins-mlpack2> Project docker ensmallen nightly build build #18: ABORTED in 2 hr 30 min: http://ci.mlpack.org/job/docker%20ensmallen%20nightly%20build/18/
< jenkins-mlpack2> Project docker mlpack nightly build build #459: STILL UNSTABLE in 2 hr 53 min: http://ci.mlpack.org/job/docker%20mlpack%20nightly%20build/459/
jeffin143 has joined #mlpack
ImQ009 has joined #mlpack
MLJens has joined #mlpack
KimSangYeon-DGU has joined #mlpack
ssk99 has joined #mlpack
ssk99 has quit [Ping timeout: 260 seconds]
< rcurtin> ssk99: good point, if you want to open a PR for that we can merge that change, otherwise I can do it
KimSangYeon-DGU has quit [Remote host closed the connection]
MLJens has quit [Ping timeout: 260 seconds]
< jeffin143> rcurtin , can we convert arma::Mat<unsigned char> to arma::mat ?
ssk99 has joined #mlpack
< zoq> jeffin143: Will open a PR with the change.
< jeffin143> Also
< jeffin143> Zoq , I am currently working on building the binding for load and same so if you don't mind , I can just make the changes
< jeffin143> Also , the tutorial mentioned, flipvertical Param , but couldn't find that anywhere
< zoq> jeffin143: If you like you can include that change in the PR you are working on.
< zoq> jeffin143: Ahh, nice catch, this should be transpose.
< zoq> jeffin143: Yes and line 155 should use the eT as well.
< jeffin143> You ment 115 instead of 155 right
< zoq> jeffin143: I see, we can remove the parameter than.
< zoq> 115 right
< jeffin143> Ok may be also I can remove that , I am not sure if that is there or not , if not there I will just remove it
< ssk99> Hey rcurtin I opened a PR for including that CMAKE flag in the list of flags. It's my first PR so let me know if everything's fine :)
xiaohong has joined #mlpack
xiaohong has quit [Remote host closed the connection]
< jeffin143> zoq : u still there ?
< zoq> jeffin143: yes
< jeffin143> The matrix is empty , and next line we are saving that matrix ,to an image
< jeffin143> And also subsequently ,what is the meaning of line no 392 and 393..???
ssk99 has quit [Remote host closed the connection]
< zoq> 392 + 393, does bring the image in the form we require for multiple images, which is one image per col. But I can't really make sense of 388.
< zoq> Looks like there are a couple of issues that I haven't seen in the first place.
< zoq> I think it makes sense to open a PR that a specific to the image loading/saving changes.
< zoq> I can do that unless you already have most of the changes ready?
< jeffin143> Is it matrix.setsize of matrix.resize ?
< zoq> jeffin143: set size, but I think img and matrix is swapped, since matrix is the input parameter and should contain the image data.
< jeffin143> Also , should it be img=matrix.col(0) ?
< jeffin143> Isntead of matrix.col(0) ? Image..??
< jeffin143> Ok I will just check and let you know..???
< zoq> jeffin143: Personally, I would remove the resize and let the user do it if necessary: https://gist.github.com/zoq/504adca4cf7bbfb6854ccc7b309a5f75
< jeffin143> We are already providing that isn't it
< jeffin143> We can specify the image height width and channels
< jeffin143> And that is resized
< zoq> jeffin143: Not sure what you mean.
< jeffin143> Umm , zoq , I will will make some changes and raise a pR , and then if you want you can just make apt changes over that
< jeffin143> Also , unsigned char image = matrix.memptr() throws and error
< jeffin143> If matrix is double..???
< jeffin143> I mean arma mat
< zoq> jeffin143: Can you post the complete line, or I guess I can also comment on the PR, once opened.
< jeffin143> The lines throws error , cannot convert double* to unsigned char in intialisation
< zoq> jeffin143: Right, that once needs a conv_to if you don't pass unsigned int
< jeffin143> Ok
< jeffin143> arma::Mat<unsigned char> image = conv_to<unsigned char>::from(matrix) ?
< jeffin143> Zoq , is above line ok.?
< zoq> jeffin143: arma::Mat<unsigned char> image = conv_to<arma::Mat<unsigned char> >::from(matrix).
< jeffin143> Ohhh
< jeffin143> Ok
< rcurtin> ssk99: saw the PR, thanks for opening that! I'll try to respond later today
jeffin143 has quit [Ping timeout: 245 seconds]
ImQ009 has quit [Quit: Leaving]
travis-ci has joined #mlpack
< travis-ci> mlpack/mlpack#7992 (master - a6040b7 : Marcus Edel): The build has errored.
travis-ci has left #mlpack []