--- Log opened Fri Jun 01 00:00:31 2018 | ||
-!- iglesias [~iglesias@f119189.upc-f.chello.nl] has quit [Ping timeout: 240 seconds] | 02:57 | |
-!- shogun-buildbot [~shogun-bu@7nn.de] has quit [Ping timeout: 260 seconds] | 03:07 | |
-!- shogun-buildbot [~shogun-bu@7nn.de] has joined #shogun | 03:09 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4309 merged by vigsterkr | 04:36 |
---|---|---|
@sukey | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/e1f02a5f49f67cede1dd3ba76a4d0710d1b3a84c by vigsterkr | 04:36 |
-shogun-buildbot:#shogun- Build trusty - libshogun - viennacl #501 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/6/builds/501 | 04:40 | |
-shogun-buildbot:#shogun- Build nightly stretch deb #122 is complete: Failure [failed dput (failure)] - http://buildbot.shogun-toolbox.org:8080/#builders/38/builds/122 | 05:12 | |
-shogun-buildbot:#shogun- Build deb1 - libshogun #472 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/10/builds/472 | 05:36 | |
-shogun-buildbot:#shogun- Build trusty - libshogun #406 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/33/builds/406 | 05:41 | |
-shogun-buildbot:#shogun- Build FC23 - libshogun #506 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/27/builds/506 | 05:41 | |
-shogun-buildbot:#shogun- Build yakkety - libshogun #498 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/1/builds/498 | 05:43 | |
-shogun-buildbot:#shogun- Build zesty - libshogun #494 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/13/builds/494 | 05:43 | |
-shogun-buildbot:#shogun- Build xenial - libshogun #495 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/16/builds/495 | 05:46 | |
-shogun-buildbot:#shogun- Build deb3 - interfaces #360 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/37/builds/360 | 06:20 | |
-!- mikeling [uid89706@gateway/web/irccloud.com/x-cftmbciyshhazfyz] has joined #shogun | 08:30 | |
-!- iglesias [~iglesias@f119189.upc-f.chello.nl] has joined #shogun | 08:35 | |
-!- iglesias [~iglesias@f119189.upc-f.chello.nl] has quit [Client Quit] | 08:35 | |
-!- HeikoS [~heiko@host86-128-122-106.range86-128.btcentralplus.com] has joined #shogun | 09:59 | |
-!- mode/#shogun [+o HeikoS] by ChanServ | 09:59 | |
-!- mikeling [uid89706@gateway/web/irccloud.com/x-cftmbciyshhazfyz] has quit [Quit: Connection closed for inactivity] | 10:39 | |
@wiking | yeeee! mike! | 10:43 |
@wiking | :) | 10:43 |
-!- shubham808 [0e8bf0fb@gateway/web/cgi-irc/kiwiirc.com/ip.14.139.240.251] has joined #shogun | 10:49 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4321 assigned to: vigsterkr by vigsterkr | 10:50 |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4321 unassigned vigsterkr by vigsterkr | 10:50 |
@wiking | shubham808, if you happene to have any questions/problems with 4321 then wuwei could help you out as he already was involved in this task in other parts of the lib. | 10:57 |
shubham808 | yes sure | 10:58 |
shubham808 | i am still looking into %newobject | 10:58 |
@wiking | kk | 10:59 |
-!- HeikoS [~heiko@host86-128-122-106.range86-128.btcentralplus.com] has quit [Quit: Leaving.] | 11:13 | |
-!- durovo [~durovo@31.8f.559e.ip4.static.sl-reverse.com] has quit [Remote host closed the connection] | 11:31 | |
-!- durovo [~durovo@31.8f.559e.ip4.static.sl-reverse.com] has joined #shogun | 11:31 | |
@wiking | wuwei, here | 11:51 |
@wiking | ? | 11:51 |
wuwei | wiking: hi | 11:51 |
@wiking | why is this necesarry sorry i missed the discussion | 11:51 |
@wiking | auto result = wrap(cross->evaluate()); | 11:51 |
@wiking | auto mean = wrap(result->get<float64_t>("mean")); | 11:51 |
@wiking | auto stddev = wrap(result->get<float64_t>("std_dev")); | 11:51 |
@wiking | (from a metaexample of cpp) | 11:52 |
@wiking | meaning wrap(...) | 11:52 |
wuwei | that's translation rule | 11:53 |
@wiking | i mean in the last 2 case it's a wrap around a simple | 11:53 |
@wiking | primiteive | 11:53 |
wuwei | so even trivial type is wrapped | 11:53 |
@wiking | i dont get why do we need to wrap a primitive | 11:53 |
@wiking | yeah but this looks super ugly | 11:53 |
@wiking | and i guess it doesnt do anything | 11:53 |
@wiking | in case of a primitive type | 11:53 |
@wiking | right? | 11:53 |
@wiking | :) | 11:53 |
wuwei | type var = ... | 11:53 |
@wiking | it's just a NAP | 11:53 |
@wiking | *NOP | 11:53 |
wuwei | This expression is always wrapped in cop | 11:53 |
wuwei | cop | 11:53 |
@wiking | where's the wrap defined? | 11:54 |
wuwei | cpp | 11:54 |
@wiking | i mean the function itself | 11:54 |
@wiking | *something* wrap(....) | 11:54 |
wuwei | some.h | 11:54 |
@wiking | i guess it's a template function or something | 11:54 |
@wiking | k | 11:55 |
@wiking | yeah i'm still not convinced about this :() | 11:55 |
@wiking | i'll have a thought around it :) | 11:56 |
wuwei | maybe we can do some tricks in meta example parser so that primitive types are recognized | 11:57 |
@wiking | yeah something like that | 11:57 |
@wiking | :) | 11:57 |
@sukey | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/d8873e68f297a80bc3ce08b5e73f48a754e2bdac by vigsterkr | 11:59 |
shubham808 | wiking: hey | 12:02 |
@wiking | ho | 12:02 |
shubham808 | for the %newobject issue | 12:03 |
@wiking | yes | 12:03 |
shubham808 | what we could do is like %newobject distance(...) right there in factory.i | 12:03 |
shubham808 | i think ? | 12:03 |
@wiking | yes | 12:03 |
@wiking | indeed | 12:03 |
shubham808 | okay | 12:03 |
@wiking | for all the factory methods | 12:03 |
@wiking | which creates a new obj | 12:03 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4322 opened by shubham808 | 12:03 |
@wiking | that shall be done :) | 12:03 |
-!- wiking [~wiking@huwico/staff/wiking] has left #shogun ["Leaving"] | 12:04 | |
-!- wiking [~wiking@huwico/staff/wiking] has joined #shogun | 12:04 | |
-!- mode/#shogun [+o wiking] by ChanServ | 12:04 | |
@wiking | miju | 12:04 |
shubham808 | wiking: ^ | 12:04 |
@wiking | shubham808, yep have you tested this locally? | 12:04 |
shubham808 | its building okay.. but i am still doubtful | 12:05 |
@wiking | k | 12:05 |
shubham808 | lets see what ci says | 12:05 |
@wiking | okok | 12:05 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4322 | 12:11 |
-!- HeikoS [~heiko@untrust-out.swc.ucl.ac.uk] has joined #shogun | 12:23 | |
-!- mode/#shogun [+o HeikoS] by ChanServ | 12:23 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4283 closed by karlnapf | 12:30 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4287 merged by karlnapf | 12:30 |
@sukey | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/f14235704a0dd7c98b4569daae771378128efdd1 by karlnapf | 12:30 |
@HeikoS | shubham808: cool, whats next? | 12:32 |
shubham808 | next i am fixing the new perceptron test | 12:33 |
shubham808 | then will look into jinja2 pr | 12:33 |
@HeikoS | shubham808: cool! | 12:34 |
@HeikoS | wiking: shubham808 will cherry pick and extend michele patch on dropping jinja from the serialization tests | 12:34 |
@HeikoS | since he will build on it for model pausing tests | 12:34 |
@wiking | HeikoS, you need to take care of windows errors | 12:35 |
@HeikoS | yep | 12:35 |
@HeikoS | thats first step | 12:35 |
@HeikoS | shubham808: ^ | 12:35 |
shubham808 | yeah | 12:36 |
@HeikoS | shubham808: let me know if you need help | 12:50 |
shubham808 | sure :) | 12:50 |
shubham808 | HeikoS: hey | 13:00 |
-shogun-buildbot:#shogun- Build deb4 - python3 #443 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/28/builds/443 | 13:00 | |
@HeikoS | shubham808: whats up? | 13:00 |
shubham808 | about EXPECT_TRUE(perceptron_initialized->get_w().equals(weights)) | 13:00 |
@HeikoS | ah yes | 13:00 |
@HeikoS | you see the difference? | 13:01 |
shubham808 | it is passing even for vectors that are not equal | 13:01 |
@HeikoS | between the implementation of operator== and .equals? | 13:01 |
@HeikoS | really? | 13:01 |
shubham808 | mmh | 13:01 |
shubham808 | let me build an example for u | 13:01 |
@HeikoS | shubham808: I would be quite surprised | 13:02 |
@HeikoS | check for bool SGVector<real_t>::equals(const SGVector<real_t>& other) const | 13:02 |
@HeikoS | in SGVector.cpp | 13:02 |
@HeikoS | there you see the implementation | 13:02 |
shubham808 | https://pastebin.com/ZvpNjVA4 | 13:06 |
shubham808 | i tried giving it wrong weights values | 13:06 |
shubham808 | (with a lot of print statement for debugging) | 13:07 |
shubham808 | https://pastebin.com/2qEkKwXc | 13:07 |
@HeikoS | shubham808: check | 13:07 |
@HeikoS | ing | 13:07 |
shubham808 | in the test log see from line 25 | 13:08 |
@HeikoS | the code doesnt seem to match the output | 13:10 |
@HeikoS | PW | 13:10 |
@HeikoS | and pw | 13:10 |
@HeikoS | in the output there is a PW | 13:10 |
@HeikoS | and in the code there only is a pw | 13:10 |
shubham808 | yeah the PW is from inside Perceptron.cpp | 13:11 |
@HeikoS | shubham808: ok i see | 13:11 |
@HeikoS | so lets see | 13:11 |
@HeikoS | first q I have, why doesnt it converge? :D | 13:11 |
shubham808 | because we gave wrong weights and set max iters to 1 | 13:12 |
@HeikoS | w probably changes during training | 13:12 |
@HeikoS | can you print both vectors just before the EXPECT_EQ call? | 13:12 |
@HeikoS | use w.display_vector() | 13:12 |
shubham808 | okay | 13:12 |
shubham808 | i see i see | 13:14 |
shubham808 | ofcourse its equal | 13:15 |
@HeikoS | lol :D | 13:18 |
@HeikoS | always remember that copying SGVector or assigning it by value doesnt copy/replicate the underying memory | 13:18 |
shubham808 | yeah | 13:19 |
@HeikoS | so now my q again | 13:19 |
@HeikoS | if you initialize with a vector from a previously converged run | 13:20 |
@HeikoS | why doesnt it converge after one iteration? | 13:20 |
shubham808 | becuase i botched it in between (to see if it fails) | 13:20 |
shubham808 | w.set_const(1.2) | 13:20 |
@HeikoS | ah ok | 13:21 |
@HeikoS | sure | 13:21 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4320 synchronized by shubham808 | 13:23 |
@HeikoS | shubham808: it all works now? | 13:24 |
shubham808 | yeah | 13:24 |
shubham808 | i manually tested it all now | 13:24 |
@HeikoS | shubham808: I just realised that maybe the bool converged should be a member of iterative machines | 13:25 |
@HeikoS | so that a user can test whether train has converged | 13:25 |
@HeikoS | but we can do that later | 13:25 |
@HeikoS | for now the PR is good | 13:25 |
shubham808 | that is useful | 13:25 |
shubham808 | yeah | 13:25 |
@HeikoS | shubham808: it is actually part of a good black box thing | 13:27 |
@HeikoS | shubham808: but ok | 13:27 |
@HeikoS | let's do the test next | 13:27 |
shubham808 | yeah | 13:27 |
@HeikoS | shubham808: let's talk here :) | 13:28 |
shubham808 | i am leaving for a bit | 13:28 |
shubham808 | lets look at the jinja pr later? | 13:29 |
@HeikoS | sure | 13:29 |
shubham808 | bye | 13:29 |
@HeikoS | see you! | 13:29 |
-!- shubham808 [0e8bf0fb@gateway/web/cgi-irc/kiwiirc.com/ip.14.139.240.251] has quit [Quit: http://www.kiwiirc.com/ - A hand crafted IRC client] | 13:29 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4322 synchronized by shubham808 | 15:35 |
-!- shubham808 [0e8bf0fb@gateway/web/cgi-irc/kiwiirc.com/ip.14.139.240.251] has joined #shogun | 15:35 | |
shubham808 | HeikoS: hey | 16:11 |
@HeikoS | shubham808: hi | 16:11 |
shubham808 | we make it like %template(kernel) shogun::kernel<float64_t>; right ? | 16:12 |
@HeikoS | no I think you just have to give explicit template parameters in %newobject | 16:15 |
@HeikoS | the template is already instantiated | 16:15 |
shubham808 | i see | 16:16 |
shubham808 | we wont need to do this for features factory | 16:18 |
shubham808 | right ? | 16:18 |
wuwei | hi, i just realized a problem | 16:22 |
wuwei | if you add %newobject to factory, there will be leak actually, imo | 16:22 |
wuwei | because currently create_object ref the created object | 16:22 |
@HeikoS | wuwei: might be | 16:28 |
@HeikoS | though there once was some convention that if you return a new object in shogun, you always REF it | 16:29 |
@HeikoS | not sure what is the state of that, needs checking | 16:29 |
@HeikoS | wuwei: iirc all the meta examples leak atm ::) | 16:29 |
wuwei | ahh..that's too bad | 16:30 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4322 synchronized by shubham808 | 16:33 |
-!- mikeling [uid89706@gateway/web/irccloud.com/x-icelohqzydpdbxjq] has joined #shogun | 16:49 | |
shubham808 | HeikoS: i made an update | 16:58 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4317 synchronized by vinx13 | 17:17 |
@wiking | mikeling, hey! how is it going? | 17:20 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4317 synchronized by vinx13 | 17:21 |
@HeikoS | shubham808: let me check | 17:42 |
@HeikoS | wuwei: if you manage to fix some of this, it would definitely count as one of the weekly meta example patches | 17:43 |
@HeikoS | wuwei: so only need to write a little cookbook in that case :) | 17:43 |
wuwei | heiko: which one? | 17:44 |
@HeikoS | wuwei: the leaks I mean | 18:41 |
wuwei | sure | 18:42 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4322 synchronized by shubham808 | 18:46 |
-!- shubham808 [0e8bf0fb@gateway/web/cgi-irc/kiwiirc.com/ip.14.139.240.251] has quit [Quit: http://www.kiwiirc.com/ - A hand crafted IRC client] | 19:00 | |
-!- shubham808 [0e8bf0fb@gateway/web/cgi-irc/kiwiirc.com/ip.14.139.240.251] has joined #shogun | 19:00 | |
-!- mikeling [uid89706@gateway/web/irccloud.com/x-icelohqzydpdbxjq] has quit [Quit: Connection closed for inactivity] | 19:09 | |
shubham808 | HeikoS: state update pr is ready | 19:20 |
shubham808 | (the perceptron one) | 19:21 |
-!- shubham808 [0e8bf0fb@gateway/web/cgi-irc/kiwiirc.com/ip.14.139.240.251] has quit [Quit: http://www.kiwiirc.com/ - A hand crafted IRC client] | 19:23 | |
-!- shubham808 [0e8bf0fb@gateway/web/cgi-irc/kiwiirc.com/ip.14.139.240.251] has joined #shogun | 19:24 | |
-!- shubham808 [0e8bf0fb@gateway/web/cgi-irc/kiwiirc.com/ip.14.139.240.251] has quit [Quit: http://www.kiwiirc.com/ - A hand crafted IRC client] | 19:34 | |
-!- HeikoS [~heiko@untrust-out.swc.ucl.ac.uk] has quit [Ping timeout: 240 seconds] | 19:43 | |
-!- durovo3 [~durovo@31.8f.559e.ip4.static.sl-reverse.com] has joined #shogun | 21:08 | |
-!- durovo [~durovo@31.8f.559e.ip4.static.sl-reverse.com] has quit [Remote host closed the connection] | 21:11 | |
-!- HeikoS [~heiko@host86-128-122-106.range86-128.btcentralplus.com] has joined #shogun | 21:13 | |
-!- mode/#shogun [+o HeikoS] by ChanServ | 21:13 | |
-!- HeikoS [~heiko@host86-128-122-106.range86-128.btcentralplus.com] has quit [Ping timeout: 244 seconds] | 22:06 | |
--- Log closed Sat Jun 02 00:00:32 2018 |
Generated by irclog2html.py 2.10.0 by Marius Gedminas - find it at mg.pov.lt!