--- Log opened Mon Jul 23 00:00:20 2018 | ||
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4373 synchronized by shubham808 | 11:29 |
---|---|---|
-!- travis-ci [~travis-ci@ec2-54-159-251-131.compute-1.amazonaws.com] has joined #shogun | 12:08 | |
travis-ci | it's Shubham Shukla's turn to pay the next round of drinks for the massacre he caused in shubham808/shogun: https://travis-ci.org/shubham808/shogun/builds/407058299 | 12:08 |
-!- travis-ci [~travis-ci@ec2-54-159-251-131.compute-1.amazonaws.com] has left #shogun [] | 12:08 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4381 synchronized by karlnapf | 12:32 |
-!- witness_ [uid10044@gateway/web/irccloud.com/x-mljwypinpznxddea] has joined #shogun | 12:46 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4381 synchronized by karlnapf | 12:52 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4381 merged by karlnapf | 13:02 |
@sukey | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/1be1dbbf6fdce20e73901d205cee253f2d4cd873 by karlnapf | 13:02 |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4382 karlnapf added label: "good first issue" | 13:08 |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4382 opened by karlnapf | 13:08 |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4382 karlnapf added label: "meta examples" | 13:08 |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4382 karlnapf added label: "bugfixing" | 13:08 |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4382 karlnapf added label: "BUG" | 13:08 |
-!- HeikoS [~heiko@2a00:23c5:e10a:5c00:51cf:933e:719e:8140] has joined #shogun | 13:18 | |
-!- mode/#shogun [+o HeikoS] by ChanServ | 13:18 | |
wuwei[m] | HeikoS: hi | 13:55 |
-!- travis-ci [~travis-ci@ec2-54-162-74-242.compute-1.amazonaws.com] has joined #shogun | 13:58 | |
travis-ci | it's Heiko Strathmann'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/407094344 | 13:58 |
-!- travis-ci [~travis-ci@ec2-54-162-74-242.compute-1.amazonaws.com] has left #shogun [] | 13:58 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4380 synchronized by vinx13 | 14:35 |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4383 opened by karlnapf | 14:51 |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4383 karlnapf added label: "good first issue" | 14:51 |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4383 karlnapf added label: "Cleanups" | 14:51 |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/4383 karlnapf added label: "testing" | 14:51 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4373 synchronized by shubham808 | 15:19 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4373 synchronized by shubham808 | 16:07 |
-!- travis-ci [~travis-ci@ec2-54-162-74-242.compute-1.amazonaws.com] has joined #shogun | 16:26 | |
travis-ci | it's Shubham Shukla's turn to pay the next round of drinks for the massacre he caused in shubham808/shogun: https://travis-ci.org/shubham808/shogun/builds/407166849 | 16:26 |
-!- travis-ci [~travis-ci@ec2-54-162-74-242.compute-1.amazonaws.com] has left #shogun [] | 16:26 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4384 opened by karlnapf | 16:46 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4384 | 16:46 |
-!- witness_ [uid10044@gateway/web/irccloud.com/x-mljwypinpznxddea] has quit [Quit: Connection closed for inactivity] | 17:34 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4380 synchronized by vinx13 | 18:34 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4380 synchronized by vinx13 | 18:38 |
@HeikoS | lisitsyn: jojo | 18:57 |
@HeikoS | lisitsyn: random q | 18:58 |
@HeikoS | shall shogun algorithms that require centered data center it themselves | 18:58 |
@HeikoS | or shall they assume that it is centered | 18:58 |
lisitsyn | HeikoS: hey | 18:58 |
@HeikoS | there are a lot of cases | 18:58 |
@HeikoS | where things are centered by the algorithm | 18:58 |
@HeikoS | and then tehre are other cases, where this is not done | 18:59 |
@HeikoS | and it is kind of up to the user to ensure this | 18:59 |
@HeikoS | (no check) | 18:59 |
lisitsyn | that's interesting | 18:59 |
@HeikoS | it is quite redundant to center in every algo | 18:59 |
@HeikoS | viktor and I also discussed sometrhign like | 18:59 |
lisitsyn | centering is sample-dependent | 18:59 |
@HeikoS | every algorithm "knows" whether it assumes centered data or not | 18:59 |
@HeikoS | lisitsyn: sure I mean it is alwazs optional | 18:59 |
lisitsyn | auto-centering is like overfitting sometimes | 18:59 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4380 synchronized by vinx13 | 19:00 |
@HeikoS | lisitsyn: yeah sure | 19:00 |
@HeikoS | but I am wondering what to do API wise | 19:00 |
@HeikoS | lisitsyn: it is optional | 19:00 |
@HeikoS | but shall the algorithms offer it? | 19:00 |
@HeikoS | alternatively we could do this in a base class | 19:01 |
@HeikoS | like in CMachine::train | 19:01 |
lisitsyn | I am not sure, I'd expect user to do that | 19:01 |
@HeikoS | if (expects_centered_data) { center_data() } | 19:01 |
@HeikoS | there is also computational issues | 19:01 |
@HeikoS | batch mode vs iterative mode | 19:01 |
@HeikoS | like say covariance matrix | 19:01 |
@HeikoS | can be done fast if I have a matrix | 19:02 |
@HeikoS | X * X^T | 19:02 |
@HeikoS | but then I need to store the matrix | 19:02 |
@HeikoS | whereas if I center on the fly | 19:02 |
@HeikoS | I dont need to | 19:02 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4380 synchronized by vinx13 | 19:02 |
@HeikoS | that is also slower | 19:02 |
@HeikoS | but in some cases (subsets active), we need to do that in order to avoid copying thigns around) | 19:02 |
@HeikoS | giving it all to the user is also fine | 19:02 |
@HeikoS | lisitsyn: more thoughts on that? | 19:03 |
@HeikoS | we can also force the user | 19:03 |
@HeikoS | like | 19:03 |
@HeikoS | she needs to set a flag | 19:03 |
lisitsyn | I am not sure how can you check if data is centered | 19:04 |
@HeikoS | model.put("YES_I_CENTERED_THE_DATA", True) | 19:04 |
@HeikoS | otherwise error in .train() | 19:04 |
@HeikoS | lisitsyn: | 19:04 |
lisitsyn | HeikoS: heh well | 19:05 |
@HeikoS | lisitsyn: I also tend to pass this to the user | 19:06 |
@HeikoS | via pipeline or so | 19:06 |
lisitsyn | yeah | 19:06 |
@HeikoS | lisitsyn: but force the user to acknowledge? | 19:07 |
lisitsyn | yeah somehow | 19:08 |
@HeikoS | lisitsyn: one could also allow the user to enable enforcements | 19:08 |
@HeikoS | i.e. | 19:08 |
@HeikoS | m.put("pls enforce", True) | 19:08 |
lisitsyn | well some checklist would be funny | 19:08 |
@HeikoS | and if that is true | 19:08 |
@HeikoS | the model checks | 19:08 |
lisitsyn | I have never seen that | 19:08 |
lisitsyn | but why not | 19:08 |
@HeikoS | okok | 19:08 |
@HeikoS | future | 19:08 |
@HeikoS | for now | 19:08 |
lisitsyn | this could probably be the best way | 19:08 |
@HeikoS | user has to be careful | 19:08 |
@HeikoS | makes the algorithsm easier as well | 19:09 |
@HeikoS | can remove all that centering code | 19:09 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4377 synchronized by vinx13 | 19:09 |
-!- iglesiasg [~iglesias@f119189.upc-f.chello.nl] has joined #shogun | 21:22 | |
-!- iglesiasg [~iglesias@f119189.upc-f.chello.nl] has quit [Remote host closed the connection] | 21:30 | |
-!- iglesiasg [~iglesias@f119189.upc-f.chello.nl] has joined #shogun | 21:30 | |
-!- iglesiasg [~iglesias@f119189.upc-f.chello.nl] has quit [Remote host closed the connection] | 21:33 | |
-!- iglesiasg [~iglesias@f119189.upc-f.chello.nl] has joined #shogun | 21:34 | |
-!- iglesiasg [~iglesias@f119189.upc-f.chello.nl] has quit [Remote host closed the connection] | 21:35 | |
-!- iglesiasg [~iglesias@f119189.upc-f.chello.nl] has joined #shogun | 21:35 | |
-!- iglesiasg [~iglesias@f119189.upc-f.chello.nl] has quit [Ping timeout: 264 seconds] | 22:15 | |
-!- iglesiasg [~iglesias@f119189.upc-f.chello.nl] has joined #shogun | 22:24 | |
-!- iglesiasg [~iglesias@f119189.upc-f.chello.nl] has quit [Remote host closed the connection] | 23:53 | |
-!- iglesiasg [~iglesias@f119189.upc-f.chello.nl] has joined #shogun | 23:54 | |
--- Log closed Tue Jul 24 00:00:22 2018 |
Generated by irclog2html.py 2.10.0 by Marius Gedminas - find it at mg.pov.lt!