IRC logs of #shogun for Wednesday, 2018-04-04

--- Log opened Wed Apr 04 00:00:09 2018
-!- witness [uid10044@gateway/web/irccloud.com/x-kznznfieqepqevbn] has joined #shogun07:07
-!- witness [uid10044@gateway/web/irccloud.com/x-kznznfieqepqevbn] has quit [Quit: Connection closed for inactivity]09:17
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4227 synchronized by grig-guz09:50
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/422709:53
-!- HeikoS [~heiko@host86-171-158-155.range86-171.btcentralplus.com] has joined #shogun11:28
-!- mode/#shogun [+o HeikoS] by ChanServ11:28
@HeikoSlisitsyn: jojo11:33
lisitsynHeikoS: hey11:33
@HeikoSlisitsyn: long time no see11:33
@HeikoSall good?11:33
lisitsynHeikoS: yeah but super busy11:33
lisitsynend of Q11:33
@HeikoSI see11:34
@HeikoSlisitsyn: I wanted to catch you back on that clone stuff11:34
@HeikoSarrays of SGO11:34
lisitsynahh yeah I am desperate to touch it as well11:34
lisitsyn:D11:34
lisitsynI think I will have some time for that this saturday11:34
@HeikoScool11:34
@HeikoSI have a bunch of local edits11:35
lisitsynHeikoS: will we plan our belgrade thing?11:35
@HeikoStests and stuff11:35
@HeikoSlisitsyn: yeah we should I guess11:35
@HeikoSlisitsyn: I mean I wrote the proposal, we can just orient ourselves on that11:35
lisitsynah ok11:35
@HeikoSlisitsyn: shall I send it again?11:37
lisitsynHeikoS: no I will find that11:38
lisitsynHeikoS: can you guide me what else is missed in the parameter thingy?11:43
lisitsynso clone of sgo arrays, what else?11:44
@HeikoSlisitsyn: globally zou mean?11:44
@HeikoSah11:44
@HeikoSI dont know of anything right now11:44
@HeikoSclean up maybe11:44
lisitsynoh ok11:44
@HeikoSonce clone is in11:44
@HeikoSthen a lot of code can go11:44
@HeikoSand then next we could look at model selection stuff11:44
@HeikoSmeta examples are ready to be transformed as well11:45
@HeikoSso then we can look at some<> integration11:45
@HeikoSbut there is a bug in swig ruby right now11:45
@HeikoSthat blocks switching all examples to the new style11:45
@HeikoSI can of course just disable the ruby examples, but would need to discuss that11:45
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4074 closed by karlnapf11:47
lisitsynHeikoS: what's the bug?11:48
@HeikoShttps://github.com/shogun-toolbox/shogun/issues/417711:49
-!- Jinquan [b7d32228@gateway/web/freenode/ip.183.211.34.40] has joined #shogun11:49
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/417711:50
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/417711:50
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/417711:51
@HeikoSlisitsyn: I updated it a bit11:51
lisitsynHeikoS: ok something similar to what we had11:52
@HeikoSlisitsyn: debugging inside the swig wrapper reveals that if you do11:52
@HeikoSsvm.set_kernel(matrix)11:52
lisitsynwith all the stuff11:52
@HeikoSthen the wrapper thinks the passed parameter is sgobject, then tries to convert and fails11:52
@HeikoSthis is only ruby11:52
lisitsynis the wrapper generated only for sgobject?11:52
@HeikoSno11:52
@HeikoSthe matrix wrapper is there as well11:52
lisitsynbut it tries sgobject first?11:52
@HeikoSyes11:53
lisitsynheh11:53
@HeikoSit checks if parameter is sgobject (that is swig genereted code)11:53
lisitsynHeikoS: sed s/../.. as cmake step11:53
lisitsyn:D11:53
@HeikoSand then the answer is "yes"11:53
@HeikoSand then it goes down the rabbithole11:53
lisitsynkrazy11:53
@HeikoShttps://github.com/swig/swig/issues/119911:53
@HeikoSI reported to swig11:53
@HeikoSthey asked for a minimal case11:53
lisitsynyeah it is always not easy11:54
@HeikoSso the q is11:54
@HeikoSshall we disable ruby examples  in test and move on?11:54
@HeikoSor shall we fix first :)11:54
lisitsynI don't know11:54
lisitsynI'd move on because11:54
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4228 merged by karlnapf11:55
@sukey[https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/8f8347a290ca932d2ea77e2b0155d9c977cbde18 by karlnapf11:55
lisitsynwell ruby+shogun is used by negative amount of users11:55
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3889 closed by karlnapf11:55
@HeikoSlisitsyn: yes11:55
@HeikoSwhile the old api is still there, users could just use that11:55
@HeikoS...11:55
lisitsynusers11:55
lisitsyn:D11:55
@HeikoSyeah11:55
@HeikoSok then11:55
lisitsynshogun+python is something >011:55
lisitsynbut shogun and ruby11:55
@HeikoSI can then refactor all meta examples to the new api11:56
@HeikoSthis would help with some<> right?11:56
lisitsynyes11:56
lisitsynwe can get back to that PR11:56
lisitsynby toni11:56
@HeikoSlisitsyn: you have a list of steps that we need to get some<> going?11:56
lisitsynHeikoS: I remember parametric ctors were the only blocker11:56
lisitsynbut who knows11:56
@HeikoSlisitsyn:  in examples or everywhere?11:57
-!- Jinquan [b7d32228@gateway/web/freenode/ip.183.211.34.40] has quit [Quit: Page closed]11:57
lisitsynHeikoS: we should stop using parameteric ctors IIRC11:57
@HeikoSeverywhere11:57
@HeikoS?11:58
lisitsynyes11:58
lisitsynbecause of renaming CLinearSVM to Some<CLInearSVM>11:58
@HeikoSlisitsyn: wanna make an entrance task for that?11:59
@HeikoSthat is a huge refactoring task11:59
-!- shubham808 [0e8bf0fb@gateway/web/freenode/ip.14.139.240.251] has joined #shogun11:59
@HeikoSand we can ask the gsoc students do some of this every week :)11:59
lisitsynHeikoS: lets first check if it is the only thing left11:59
@HeikoSkk11:59
@HeikoSlisitsyn: I mean first thing is clone11:59
@HeikoSfor raw arrays11:59
lisitsynyeah11:59
lisitsynand then I will check11:59
@HeikoSonce that is in, I can do all sorts of new things11:59
lisitsynif that PR is mergeable11:59
shubham808@HeikoS there is a problem with #422912:04
@HeikoSshubham808: hi12:04
@HeikoSshubham808: what is the problem?12:05
-!- shubham808_ [2ff7812c@gateway/web/freenode/session] has joined #shogun12:06
@HeikoSshubham808_:  what is the problem?12:06
shubham808_the code that generates the results needed by those tests in now combined with respective operator functions12:06
shubham808_since we decided to get rid of the job based structure12:07
@HeikoSI see12:08
-!- shubham808 [0e8bf0fb@gateway/web/freenode/ip.14.139.240.251] has quit [Ping timeout: 260 seconds]12:08
@HeikoSas long as you test something downstream (some test that is based on the code), it should be ok then12:08
@HeikoSshubham808_: is that the case?12:09
shubham808_in rational approximation unit test we are checking final results12:09
-!- tctara_ [~quassel@irc.redcorelinux.org] has joined #shogun12:09
shubham808_so yeah12:09
shubham808_we are checking something based on the code12:10
shubham808_for all 3 operator functions12:10
@HeikoSok cool that is good12:11
@HeikoSshubham808_: would there be a way of pulling out the functionality of those operator functions into separate functions, and then test those functions?12:11
@HeikoSthat would be best12:11
shubham808_that would also involve adding set/get methods for weights and shifts12:13
shubham808_okay lets try that12:14
-!- zxtx_ [~zxtx@sunnythinking.org] has joined #shogun12:14
shubham808_but it will make the code uglier12:14
@HeikoSshubham808_: really?12:14
@HeikoSshubham808_: actually12:15
-!- Netsplit *.net <-> *.split quits: tctara, zxtx12:16
@HeikoSshubham808_: you know what, I think it is fine12:17
@HeikoSas long as you test downstream, I think we are good12:18
@HeikoSso I will merge this12:18
shubham808_yeah nice :)12:18
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4229 merged by karlnapf12:18
@sukey[https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/38a6489561968aa2b4fa391fd053fc583121304b by karlnapf12:18
@HeikoSshubham808_: nice, that was a huge cleanup, thanks for that12:18
@HeikoSshubham808_: whats next? :)12:19
shubham808_next is parallelization12:19
shubham808_then maybe i will deal with #404412:20
-!- shubham808_ [2ff7812c@gateway/web/freenode/session] has quit [Quit: Page closed]12:23
-!- travis-ci [~travis-ci@ec2-54-159-25-82.compute-1.amazonaws.com] has joined #shogun12:45
travis-ciit's Albert's turn to pay the next round of drinks for the massacre he caused in shogun-toolbox/shogun: https://travis-ci.org/shogun-toolbox/shogun/builds/36205486112:45
-!- travis-ci [~travis-ci@ec2-54-159-25-82.compute-1.amazonaws.com] has left #shogun []12:45
-!- travis-ci [~travis-ci@ec2-54-81-21-151.compute-1.amazonaws.com] has joined #shogun12:59
travis-ciit's Albert's turn to pay the next round of drinks for the massacre he caused in shogun-toolbox/shogun: https://travis-ci.org/shogun-toolbox/shogun/builds/36205486112:59
-!- travis-ci [~travis-ci@ec2-54-81-21-151.compute-1.amazonaws.com] has left #shogun []12:59
-!- travis-ci [~travis-ci@ec2-54-92-167-44.compute-1.amazonaws.com] has joined #shogun13:09
travis-ciit's Shubham Shukla's turn to pay the next round of drinks for the massacre he caused in shogun-toolbox/shogun: https://travis-ci.org/shogun-toolbox/shogun/builds/36206735213:09
-!- travis-ci [~travis-ci@ec2-54-92-167-44.compute-1.amazonaws.com] has left #shogun []13:09
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4208 merged by karlnapf13:42
@sukey[https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/63594a456919492d35a799d26b5974e9f9d4fb60 by karlnapf13:42
-!- travis-ci [~travis-ci@ec2-54-81-21-151.compute-1.amazonaws.com] has joined #shogun13:45
travis-ciit's Shubham Shukla's turn to pay the next round of drinks for the massacre he caused in shogun-toolbox/shogun: https://travis-ci.org/shogun-toolbox/shogun/builds/36206735213:45
-!- travis-ci [~travis-ci@ec2-54-81-21-151.compute-1.amazonaws.com] has left #shogun []13:45
-!- travis-ci [~travis-ci@ec2-54-92-167-44.compute-1.amazonaws.com] has joined #shogun14:40
travis-ciit's Elfarouk Yasser's turn to pay the next round of drinks for the massacre he caused in shogun-toolbox/shogun: https://travis-ci.org/shogun-toolbox/shogun/builds/36209671614:40
-!- travis-ci [~travis-ci@ec2-54-92-167-44.compute-1.amazonaws.com] has left #shogun []14:40
-!- travis-ci [~travis-ci@ec2-54-159-25-82.compute-1.amazonaws.com] has joined #shogun14:45
travis-ciit's Elfarouk Yasser's turn to pay the next round of drinks for the massacre he caused in shogun-toolbox/shogun: https://travis-ci.org/shogun-toolbox/shogun/builds/36209671614:45
-!- travis-ci [~travis-ci@ec2-54-159-25-82.compute-1.amazonaws.com] has left #shogun []14:45
-!- durovo [~durovo@cc.b7.2ea9.ip4.static.sl-reverse.com] has quit [Remote host closed the connection]15:42
-!- durovo1 [~durovo@b1.aa.36a9.ip4.static.sl-reverse.com] has joined #shogun15:42
-!- sukey [~nodebot@188.165.194.51] has quit [Remote host closed the connection]15:52
-!- sukey [~nodebot@ks312251.kimsufi.com] has joined #shogun15:52
-!- mode/#shogun [+o sukey] by ChanServ15:52
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4226 closed by ypic17:15
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4230 opened by karlnapf19:17
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4230 synchronized by karlnapf19:51
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4230 synchronized by karlnapf19:52
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/423019:53
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/423021:03
-!- witness [uid10044@gateway/web/irccloud.com/x-tcmvrhvotlzspjtc] has joined #shogun21:57
-!- HeikoS [~heiko@host86-171-158-155.range86-171.btcentralplus.com] has quit [Ping timeout: 255 seconds]22:13
--- Log closed Thu Apr 05 00:00:11 2018

Generated by irclog2html.py 2.10.0 by Marius Gedminas - find it at mg.pov.lt!