--- Log opened Wed Jul 11 00:00:03 2018 | ||
-!- HeikoS [~heiko@2a00:23c5:e10a:5c00:e59f:8ac6:d583:5195] has quit [Ping timeout: 256 seconds] | 00:44 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4352 synchronized by vinx13 | 02:56 |
---|---|---|
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4348 synchronized by vinx13 | 02:57 |
wuwei | wiking: hi | 11:02 |
-!- HeikoS [~heiko@host86-172-81-177.range86-172.btcentralplus.com] has joined #shogun | 11:13 | |
-!- mode/#shogun [+o HeikoS] by ChanServ | 11:13 | |
-!- sukey [~nodebot@ks312251.kimsufi.com] has quit [Remote host closed the connection] | 12:15 | |
-!- sukey1 [~nodebot@ks312251.kimsufi.com] has joined #shogun | 12:15 | |
sukey1 | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4356 synchronized by shubham808 | 12:34 |
@wiking | wuwei, hey | 13:11 |
@wiking | wuwei, sorry still jetlagged | 13:11 |
wuwei | hi | 13:11 |
wuwei | wiking: i think we need to discuss about plans for this and following weeks | 13:12 |
@wiking | cool | 13:12 |
@wiking | just one question https://github.com/shogun-toolbox/shogun/pull/4348 | 13:12 |
-!- travis-ci [~travis-ci@ec2-54-145-176-190.compute-1.amazonaws.com] has joined #shogun | 13:13 | |
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/402599854 | 13:13 |
-!- travis-ci [~travis-ci@ec2-54-145-176-190.compute-1.amazonaws.com] has left #shogun [] | 13:13 | |
@wiking | i would do the REQUIRE should be using different exception | 13:13 |
@wiking | than ShogunException | 13:13 |
wuwei | sure | 13:14 |
wuwei | we have MachineNotFitted, do we need TransformerNotFitted ? | 13:15 |
@HeikoS | What about "NotFittedException" | 13:16 |
wuwei | yeah cool | 13:16 |
@wiking | wuwei, other than that the feature branch seems to be ready | 13:19 |
wuwei | btw features view pr is to that feaeture branch | 13:21 |
wuwei | i thought i would be helpful to continue from factor have been made, so i opened that pr to feature/transformers | 13:22 |
@wiking | mmmm | 13:22 |
@wiking | i see | 13:24 |
@wiking | that's gonna be a bit scary :) | 13:24 |
@wiking | but it's ok for the time being | 13:24 |
@wiking | we can change the pr to be merged into a different branch | 13:24 |
@wiking | so i would like to get 4348 fixed | 13:25 |
wuwei | okay | 13:25 |
@wiking | and then merge it and then merge feature/transformers into develop | 13:25 |
wuwei | do we need to move converter/ and preproc/ to transformer/ | 13:29 |
-!- travis-ci [~travis-ci@ec2-54-145-176-190.compute-1.amazonaws.com] has joined #shogun | 13:45 | |
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/402599854 | 13:45 |
-!- travis-ci [~travis-ci@ec2-54-145-176-190.compute-1.amazonaws.com] has left #shogun [] | 13:45 | |
-!- travis-ci [~travis-ci@ec2-50-19-42-72.compute-1.amazonaws.com] has joined #shogun | 14:04 | |
travis-ci | it's Wuwei Lin'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/402346487 | 14:04 |
-!- travis-ci [~travis-ci@ec2-50-19-42-72.compute-1.amazonaws.com] has left #shogun [] | 14:04 | |
-!- travis-ci [~travis-ci@ec2-54-163-108-187.compute-1.amazonaws.com] has joined #shogun | 14:05 | |
travis-ci | it's Wuwei Lin'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/402118838 | 14:05 |
-!- travis-ci [~travis-ci@ec2-54-163-108-187.compute-1.amazonaws.com] has left #shogun [] | 14:05 | |
@wiking | wuwei, could :) | 14:12 |
@wiking | but it's not necessary | 14:12 |
@wiking | at this point imo | 14:12 |
wuwei | sure | 14:13 |
wuwei | i'll put it in the future | 14:13 |
sukey1 | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4348 synchronized by vinx13 | 14:19 |
@wiking | btw http://grammarist.com/usage/fit-fitted/ | 14:20 |
@wiking | :) | 14:20 |
wuwei | mmm | 14:22 |
@wiking | i didn't know just sharing :) | 14:22 |
@wiking | anyhow it's good | 14:22 |
@wiking | just change from check_fitted to is_fitted() | 14:22 |
@wiking | that'd be a more standard way | 14:23 |
@wiking | to query the state of the obj | 14:23 |
wuwei | haha | 14:23 |
wuwei | check_fitted may throw exception, so maybe assert_fitted is better? | 14:24 |
wuwei | it doesn't have return value | 14:24 |
@wiking | ah yeah | 14:24 |
@wiking | now i see | 14:24 |
@wiking | yeah assert_fitted() is good in this case | 14:25 |
@wiking | but the m_fitted param should be registered as is_fitted | 14:25 |
@wiking | so we can do later | 14:25 |
@wiking | get("is_fitted") | 14:25 |
wuwei | yeah | 14:25 |
@wiking | i know it'd be easier to have | 14:25 |
@wiking | get("fitted") (shorter) | 14:25 |
@wiking | but somehow i feel the is_fitted represents better what that value is about | 14:26 |
sukey1 | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4348 synchronized by vinx13 | 14:30 |
sukey1 | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4348 merged by vigsterkr | 14:31 |
sukey1 | [https://github.com/shogun-toolbox/shogun] vigsterkr pushed 6 commits: | 14:31 |
sukey1 | https://github.com/shogun-toolbox/shogun/commit/87e13afdd73a49eb8fe7dc5765144ddd22eff68c | 14:31 |
@wiking | ok | 14:31 |
sukey1 | https://github.com/shogun-toolbox/shogun/commit/54e0afae103de0bce86638cee4d8c7d6c498375a | 14:31 |
sukey1 | https://github.com/shogun-toolbox/shogun/commit/c492857d5d2066decdf6088ea4cfb9fae46dc3e0 | 14:32 |
sukey1 | https://github.com/shogun-toolbox/shogun/commit/d53dc1ca353a712d2462e6e9ee9d548514ef0884 | 14:32 |
sukey1 | https://github.com/shogun-toolbox/shogun/commit/a21872622d28fd71d1a4aa3da9f980329b48e352 | 14:32 |
sukey1 | https://github.com/shogun-toolbox/shogun/commit/a241ab14506083f1844187a2a5e62b45ba99b910 | 14:32 |
@wiking | wuwei, lets see what the CIs say and if all good i'll merge the whole branch into develop | 14:32 |
@wiking | finally :) | 14:32 |
@wiking | this is an amazing feature! | 14:32 |
@wiking | pipeline + typed exceptions + transformers! | 14:32 |
wuwei | cool! | 14:32 |
@wiking | great great stuff! | 14:32 |
wuwei | wiking: what's next? | 14:34 |
wuwei | i've been working on dense features these days | 14:35 |
wuwei | last pr is https://github.com/shogun-toolbox/shogun/pull/4365 | 14:35 |
@wiking | ok | 14:38 |
@wiking | remember that we had a small task | 14:38 |
@wiking | about CV and pipeline | 14:38 |
@wiking | i mean to have a cookbook about that | 14:38 |
@wiking | that uses PruneSubVarMean + a linear machine or something | 14:39 |
wuwei | sure | 14:39 |
wuwei | btw on-the-fly transformer currently doesn't with dense features | 14:40 |
@wiking | mmmm why? | 14:41 |
wuwei | it will need some refactor on DenseFeatures::get_feature_vector | 14:41 |
wuwei | because the original code in get_feature_vector actually doesn't call preprocessors | 14:41 |
@wiking | argh | 14:42 |
wuwei | https://github.com/shogun-toolbox/shogun/blob/f10f3ef2b28b7a5f0e137b3e4a7f8abf80a64201/src/shogun/features/DenseFeatures.cpp#L120 | 14:42 |
wuwei | it will return from this line | 14:42 |
@wiking | ouch | 14:43 |
sukey1 | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4294 closed by iglesias | 14:51 |
sukey1 | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4337 closed by iglesias | 14:52 |
-!- iglesias [503877bd@gateway/web/freenode/ip.80.56.119.189] has joined #shogun | 14:53 | |
@wiking | iglesias, why close?! | 14:55 |
@wiking | man we can still salvage things from there :) | 14:55 |
iglesias | yeah | 14:55 |
iglesias | still possible to do even if it is close ;-) | 14:55 |
iglesias | well closed | 14:55 |
iglesias | wiking: is there a garbage collector for closed PRs or something like that? :P | 14:56 |
@wiking | :> | 14:57 |
iglesias | so this is the project eventually: https://github.com/shogun-toolbox/shogun/pull/4216/files | 14:58 |
iglesias | and one or two other cookbooks iirc | 14:58 |
iglesias | wiking: we had a lunch discussion about Zurich yesterday | 15:00 |
@wiking | ? | 15:00 |
@wiking | where? | 15:01 |
iglesias | just at job | 15:01 |
@wiking | ah what was it about? :) | 15:01 |
iglesias | just sharing something I found surprising | 15:01 |
iglesias | I heard that phd student salary in Zurich ~ 72k euro / year | 15:01 |
@wiking | ye | 15:01 |
@wiking | :) | 15:01 |
iglesias | I googled it up | 15:01 |
@wiking | i mean it depends on the subject (which is crazy imo) | 15:02 |
iglesias | and someone was saying "yeah, you can also make that in fast food around here" | 15:02 |
iglesias | xD | 15:02 |
@wiking | but yeah CS phd is around that | 15:02 |
@wiking | humanities phd would get much less unfortunately | 15:02 |
@wiking | but if you are doing a hipster phd then its the case | 15:02 |
iglesias | that doesn't sound fair in principle | 15:02 |
iglesias | hahahaha hphd | 15:02 |
@wiking | yeah hphd | 15:02 |
@wiking | :) | 15:02 |
iglesias | hphd in layers | 15:03 |
@wiking | eyyeye LAYER IT! | 15:03 |
@wiking | but on the other hand that's true | 15:03 |
@wiking | that if you do basic job you woulnd't get much less | 15:03 |
iglesias | this thesis proves that we need more layers; always, more layers | 15:03 |
@wiking | which i like a lot | 15:03 |
@wiking | but still there's a quite huge gap here as well | 15:03 |
@wiking | between the lowest and the highest income :) | 15:03 |
@wiking | as usual | 15:03 |
@wiking | :> | 15:03 |
iglesias | yeah | 15:03 |
iglesias | I guess it is also the case around here | 15:04 |
@wiking | but yeah afaik a shop clerk would get around 5k/month | 15:04 |
iglesias | I see lot of Teslas around xD | 15:04 |
iglesias | especially in the High Tech Campus | 15:04 |
@wiking | but then again cost of living is high here... sooo.... | 15:04 |
@wiking | it's all about the ratios | 15:04 |
iglesias | Yes, indeed! It has to be taken together (salary + cost of living), I check numbeo for this stuff | 15:05 |
@wiking | ah yeah | 15:05 |
@wiking | there was a website | 15:05 |
@wiking | about this | 15:05 |
@wiking | :) | 15:05 |
iglesias | https://www.numbeo.com/cost-of-living/compare_cities.jsp?country1=Netherlands&country2=Germany&city1=Eindhoven&city2=Berlin | 15:06 |
iglesias | https://www.numbeo.com/cost-of-living/compare_cities.jsp?country1=Switzerland&country2=Germany&city1=Zurich&city2=Berlin | 15:06 |
iglesias | vamos todos a Berlin | 15:06 |
iglesias | where it all started xD | 15:06 |
@wiking | heheh | 15:06 |
@wiking | iglesias, ping sonney2k_ he has a lot of open position at tomtom | 15:06 |
@wiking | :) | 15:06 |
iglesias | wiking: let me show one thing I read some weeks ago | 15:07 |
iglesias | https://www.think-cell.com/en/career/jobs/development.shtml | 15:07 |
@wiking | check this iglesias https://www.numbeo.com/cost-of-living/compare_countries_result.jsp?country1=Switzerland&country2=Serbia | 15:08 |
@wiking | :D | 15:08 |
iglesias | "A competitive salary from the start and a raise to EUR 120,000 annually after only one year" | 15:08 |
iglesias | unbelievable | 15:08 |
@wiking | ah yeah so you are listening to cpp cast as well | 15:08 |
@wiking | :))) | 15:08 |
@wiking | (they always advertise there) | 15:08 |
iglesias | si | 15:09 |
@wiking | but i mean even this is crazy | 15:09 |
@wiking | https://www.numbeo.com/cost-of-living/compare_countries_result.jsp?country1=Netherlands&country2=Serbia | 15:09 |
@wiking | :) | 15:09 |
iglesias | also saw it in the meetingc++ conf website | 15:09 |
iglesias | they are one of the sponsors apparently | 15:09 |
@wiking | "Consumer Prices Including Rent in Serbia are 59.35% lower than in Netherlands" | 15:09 |
@wiking | :D | 15:09 |
iglesias | wow man the average salary is insane too | 15:09 |
@wiking | yeah | 15:10 |
@wiking | :)) | 15:10 |
@wiking | and that is inflated imo | 15:10 |
@wiking | for serbia | 15:10 |
@wiking | the avg should be around 200-250 | 15:10 |
iglesias | niet goed | 15:10 |
@wiking | but i mean if you are lucky and work in CS | 15:10 |
@wiking | and you do freelancing | 15:11 |
iglesias | $$$$$ | 15:11 |
@wiking | which most of the ppl do there whom i know | 15:11 |
@wiking | then it's really nice | 15:11 |
-!- HeikoS [~heiko@host86-172-81-177.range86-172.btcentralplus.com] has quit [Ping timeout: 264 seconds] | 15:33 | |
-!- travis-ci [~travis-ci@ec2-54-163-108-187.compute-1.amazonaws.com] has joined #shogun | 15:36 | |
travis-ci | it's Wuwei Lin'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/402639172 | 15:36 |
-!- travis-ci [~travis-ci@ec2-54-163-108-187.compute-1.amazonaws.com] has left #shogun [] | 15:36 | |
-!- HeikoS [~heiko@2a00:23c5:e10a:5c00:e59f:8ac6:d583:5195] has joined #shogun | 15:54 | |
-!- mode/#shogun [+o HeikoS] by ChanServ | 15:54 | |
-!- travis-ci [~travis-ci@ec2-54-163-108-187.compute-1.amazonaws.com] has joined #shogun | 15:55 | |
travis-ci | it's Wuwei Lin'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/402639172 | 15:55 |
-!- travis-ci [~travis-ci@ec2-54-163-108-187.compute-1.amazonaws.com] has left #shogun [] | 15:55 | |
sukey1 | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4366 opened by vinx13 | 16:25 |
lisitsyn | wiking: how do I check this feature? | 16:30 |
@wiking | whichone? | 16:31 |
lisitsyn | https://github.com/shogun-toolbox/shogun/pull/4348 | 16:31 |
lisitsyn | ah this thing | 16:31 |
@wiking | feature/transformers | 16:31 |
@wiking | has it all | 16:31 |
@wiking | :) | 16:31 |
@wiking | just trying to get that crazy amount of swig warnings | 16:32 |
@wiking | silenced | 16:32 |
@wiking | so that the CIs do not fail | 16:32 |
lisitsyn | auto auto(auto auto) { auto; } | 16:34 |
@wiking | AUTO! | 16:35 |
@wiking | we need more AUTO | 16:35 |
@wiking | :> | 16:35 |
-!- iglesias [503877bd@gateway/web/freenode/ip.80.56.119.189] has quit [Ping timeout: 252 seconds] | 16:46 | |
-!- travis-ci [~travis-ci@ec2-54-163-108-187.compute-1.amazonaws.com] has joined #shogun | 17:22 | |
travis-ci | it's Wuwei Lin'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/402639172 | 17:22 |
-!- travis-ci [~travis-ci@ec2-54-163-108-187.compute-1.amazonaws.com] has left #shogun [] | 17:22 | |
sukey1 | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4366 synchronized by vinx13 | 17:28 |
sukey1 | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4366 merged by karlnapf | 19:40 |
sukey1 | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/b1d366d7fc79ee7162c68ec0a4561c762c248902 by karlnapf | 19:40 |
-!- travis-ci [~travis-ci@ec2-54-196-235-103.compute-1.amazonaws.com] has joined #shogun | 20:31 | |
travis-ci | it's Wuwei Lin'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/402774876 | 20:31 |
-!- travis-ci [~travis-ci@ec2-54-196-235-103.compute-1.amazonaws.com] has left #shogun [] | 20:31 | |
-!- travis-ci [~travis-ci@ec2-50-19-42-72.compute-1.amazonaws.com] has joined #shogun | 20:42 | |
travis-ci | it's Wuwei Lin'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/402774876 | 20:43 |
-!- travis-ci [~travis-ci@ec2-50-19-42-72.compute-1.amazonaws.com] has left #shogun [] | 20:43 | |
--- Log closed Thu Jul 12 00:00:05 2018 |
Generated by irclog2html.py 2.10.0 by Marius Gedminas - find it at mg.pov.lt!