IRC logs of #shogun for Saturday, 2017-06-24

--- Log opened Sat Jun 24 00:00:26 2017
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3858 synchronized by geektoni00:24
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3751 synchronized by micmn00:49
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3750 closed by micmn00:50
-!- TingMiao [uid229534@gateway/web/irccloud.com/x-bhbuvmdyizfzugrh] has quit [Quit: Connection closed for inactivity]00:53
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3860 vigsterkr added label: "entrance"05:28
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3860 vigsterkr added label: "memory problems"05:28
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3860 vigsterkr added label: "BUG"05:28
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3860 opened by vigsterkr05:28
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3860 vigsterkr added label: "bugfixing"05:28
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3861 opened by vigsterkr05:31
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3862 vigsterkr added label: "bugfixing"05:32
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3862 opened by vigsterkr05:32
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3862 vigsterkr added label: "BUG"05:32
@wikinglisitsyn, ping ping07:59
-!- geektoni [~geektoni@93-34-128-38.ip49.fastwebnet.it] has joined #shogun09:45
@wikinggeektoni, hey09:45
@wikinggeektoni, it's weeend :)09:45
@wiking*weekend09:45
geektonihi wiking. yeah, I know ;) but I want to address your comments before next week.09:46
@wikingheheh09:46
@wikingok09:46
@sukey[https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/cf1f846a210ed9da4aaafb5f4760f3fb69c43ace by vigsterkr10:52
@sukey[https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/c59e42de6e728e082c818569f4fa1a282c8e3cdb by vigsterkr10:58
@sukey[https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/2252d06e2a06dceeaa9528e3b772be34aa823034 by vigsterkr11:06
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3859 synchronized by MikeLing11:18
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3859 synchronized by MikeLing11:34
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3863 opened by micmn11:41
@sukey[https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/4854bd6370d8e505d6c8fba51894a9930309e38a by vigsterkr11:45
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3860 closed by vigsterkr11:47
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3863 merged by vigsterkr11:47
@sukey[https://github.com/shogun-toolbox/shogun] vigsterkr pushed 2 commits:11:47
@sukeyhttps://github.com/shogun-toolbox/shogun/commit/1e8ab8f9656c5b2083a92ff049d013af0bbabe0911:47
@sukeyhttps://github.com/shogun-toolbox/shogun/commit/e87649f67698996bbd512b17c92edaefe539786011:47
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3751 synchronized by micmn13:07
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3858 synchronized by geektoni13:11
@sukey[https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/140a8eb77cfcd523711274789d4e5d6b5f30bd52 by vigsterkr13:30
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3843 synchronized by micmn13:33
micmnwiking: ping13:48
@wikingpong13:48
micmnwhat's the point of having a singleton with AddGlobalTestEnvironment?13:49
micmnhttps://github.com/micmn/shogun/blob/41f2ca4995d517c0729363e0f854bddb03af997c/tests/unit/base/main_unittest.cc#L7413:49
micmnhttps://github.com/micmn/shogun/blob/41f2ca4995d517c0729363e0f854bddb03af997c/tests/unit/environments/LinearTestEnvironment.h#L5213:49
@wikingeeeeeeeeeeeeeeeeeeeeeeee13:51
@wikingmicmn, i wonder why it doesn't even follow13:55
@wikingthe override of virtual void SetUp() and virtual void TearDown()13:55
@wiking:<13:55
micmnyep, I was writing the regression env and stumbled into this13:56
@wikingyeah plz fix it13:56
micmnit's a mix of singleton and gtest env13:56
@wikingand mea culpa for merging these :(13:56
@wikingyeah i can see that13:56
@wiking:(((13:56
micmnyeah you have too many to do :P13:58
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3858 synchronized by geektoni13:58
@wikingsukey, flip14:09
@sukey(/ .□.) ︵╰(゜Д゜)╯︵ /(.□. )14:09
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3844 closed by vigsterkr14:22
@sukey[https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3844 reopened by vigsterkr14:25
-!- geektoni [~geektoni@93-34-128-38.ip49.fastwebnet.it] has quit [Quit: Leaving.]14:34
-!- TingMiao [uid229534@gateway/web/irccloud.com/x-zmqavftcpyclebqv] has joined #shogun16:47
-!- mikeling [uid89706@gateway/web/irccloud.com/x-rjocwogybngdeeuj] has joined #shogun17:23
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3864 opened by micmn17:34
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3864 synchronized by micmn17:58
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/3864 merged by vigsterkr19:05
@sukey[https://github.com/shogun-toolbox/shogun] vigsterkr pushed 2 commits:19:05
@sukeyhttps://github.com/shogun-toolbox/shogun/commit/ca5a503363a84493d51a0ef7b57ef3c97c49dc3319:05
@sukeyhttps://github.com/shogun-toolbox/shogun/commit/0d87fbd80a8f1c98ca652f2c21dad5f85218c51319:05
shogun-buildbotbuild #249 of deb1 - libshogun is complete: Failure [failed test]  Build details are at http://buildbot.shogun-toolbox.org/builders/deb1%20-%20libshogun/builds/249  blamelist: Michele Mazzoni <mic.mazzoni@gmail.com>, Viktor Gal <vigsterkr@gmail.com>19:08
micmnwiking, mikeling: SVMOcas fails :S19:13
mikelingmicmn: mmm, you mean it will fails randomly or?19:16
mikelingbecause I found nothing shows up on co19:16
mikelingci19:16
micmnbuildbot19:16
micmnhttp://buildbot.shogun-toolbox.org/builders/deb1%20-%20libshogun/builds/249/steps/test/logs/stdio19:17
mikelingmicmn: oh, I see. BTW, had you read this comment by Heikos in https://github.com/shogun-toolbox/shogun/pull/3812#pullrequestreview-45765041?19:19
micmnwhich one?19:20
micmnah i see19:20
mikelingIt about to restrict Gaussian output and make the data will be linearly separable every time19:20
micmnyeah19:21
micmnare you working on that?19:21
mikelingno, haven't yet19:22
mikelingBTW, I found this https://github.com/shogun-toolbox/shogun/blob/develop/src/shogun/features/DataGenerator.cpp#L32 in CDataGenerator19:22
mikelingisn't it the same thing we are going to use in the fixture?19:23
mikelingor?19:23
mikelingI'm not sure about that either :)19:23
micmnlet me see19:23
micmnbtw isn't a bit late over there? :P19:23
mikelingit's 1:30 am if you are talking about the time right now :)19:25
micmnyep :p19:26
micmngenerate_checkboard_data seems useful19:26
micmnsince you can control class overlap19:27
mikelingmmm, so maybe we could just use it directly :P19:27
mikelingwiking Herikos^19:28
-!- travis-ci [~travis-ci@ec2-50-16-14-196.compute-1.amazonaws.com] has joined #shogun19:29
travis-ciit's Viktor Gal'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/24659042119:29
-!- travis-ci [~travis-ci@ec2-50-16-14-196.compute-1.amazonaws.com] has left #shogun []19:29
mikelingmicmn:I should read all the functions carefully before I start working on that issue.19:30
mikelingsorry for misleading you to the wrong direction in the beginning19:30
micmnoh don't worry I guess we learn both from these things19:31
mikelingmicmn: ;)19:33
mikelingmicmn So, actually I'm working on the CRandom issue. The things I do basically is going to remove the global random which is sg_rng19:35
mikelingwiking: micmn but really makes me headache thing is its usage in unittest like https://github.com/shogun-toolbox/shogun/blob/develop/tests/unit/neuralnets/DeepBeliefNetwork_unittest.cc#L4419:36
micmnwhy?19:37
mikelingit will given a seed to random generator19:38
mikelingbut the DeepBeliefNetwork won't actually use that random generator19:38
micmnhttps://github.com/shogun-toolbox/shogun/blob/develop/tests/unit/neuralnets/DeepBeliefNetwork_unittest.cc#L57 ?19:39
mikelingmicmn: how about this https://github.com/shogun-toolbox/shogun/blob/develop/tests/unit/neuralnets/NeuralNetwork_unittest.cc#L31219:40
micmnmmm isn't neural networks initialization random?19:41
mikelingyes, it is. mmm, maybe I should add a set_seed() function in neural networks19:44
mikelingok, I found it. Thank you micmn :)19:46
micmni.e. https://github.com/micmn/shogun/blob/7339764920061d35581287f2483c24cc6187901e/src/shogun/neuralnets/NeuralLinearLayer.cpp#L61-L6119:46
micmnnp19:46
micmnbtw which is the reason to remove these CRandom::seed() in unit tests?19:47
mikelingmicmn: mmm, you mean CMath::init_random()?19:54
mikelingbecause we are going to remove the global random in https://github.com/shogun-toolbox/shogun/blob/develop/src/shogun/mathematics/Math.h#L12719:56
mikelings/in/like19:56
mikelingso we no longer need random  functions in CMath19:57
micmnah!19:59
micmnand how you'll generate random numbers then?20:00
mikelingwe will introduce an variable named m_rng in SGObject20:02
mikelingand then use it inside20:02
mikelingit's a CRandom object20:02
micmnso each object has its rng?20:03
micmnand you'll do i.e. m_rng->random(0,1)20:04
mikelingthe only problem is the CRandom is  inherited from SGObject, so basically it will be a pointer like CRandom*20:04
mikelingyep20:04
micmnok got it thx :)20:04
mikelingnp ;)20:05
-!- mikeling [uid89706@gateway/web/irccloud.com/x-rjocwogybngdeeuj] has quit [Quit: Connection closed for inactivity]22:58
--- Log closed Sun Jun 25 00:00:28 2017

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