Open in new window / Try shogun cloud
--- Log opened Sun Oct 02 00:00:52 2011
blackburn1oh I was wrong with that extension00:46
-!- blackburn1 [~blackburn@188.168.4.116] has quit [Quit: Leaving.]01:42
-!- blackburn [~blackburn@188.168.3.99] has joined #shogun10:19
-!- blackburn [~blackburn@188.168.3.99] has quit [Ping timeout: 260 seconds]17:25
-!- blackburn [~blackburn@188.168.14.11] has joined #shogun19:20
@sonney2kblackburn, which extension?19:24
blackburnsonney2k: I still trying to get my kernel LTSA work right19:24
@sonney2kok19:25
blackburnsonney2k: we can't disable pthreads?19:30
@sonney2k?19:30
blackburnsonney2k: no --disable-pthreads option19:31
@sonney2kI see19:31
@sonney2kindeed...19:31
blackburnsonney2k: have you seen a cool feature I added? now possible to set target dimensionality to -1 and it will be reduced by 119:34
@sonney2kisn't hat confusing?19:35
blackburnI don't know, is it?19:36
@sonney2kwell you need to document it at least20:02
blackburnoh20:14
CIA-3shogun: Sergey Lisitsyn master * r246ccf6 / src/shogun/preprocessor/DimensionReductionPreprocessor.cpp : Fixed wrong destruction of DR preprocessor base class - http://git.io/STWLGA20:15
CIA-3shogun: Sergey Lisitsyn master * r51eeb3d / (7 files): Improved multithreading codestyle, fixes some structure - http://git.io/xdq67Q20:15
blackburnsonney2k: last one commit reduced LoC related to multithreading VERY much20:16
@sonney2kblackburn, so it is not using that weird syntax but just { } are enough?20:17
@sonney2kblackburn, the only problem I see is that it is unreadable - I mean you have like 20 args20:18
@sonney2kand it is totally unclear which one is for what20:18
blackburnsonney2k: not more unreadable than it was :D20:19
@sonney2kwell before it at least said start_idx=020:19
@sonney2know it says 0 somewhere hidden among those args...20:19
blackburnsure, but INFINITY lines of .x=y is crazy too20:19
@sonney2kno20:19
@sonney2kit helps to understand what you do and you could do that on the same line too20:20
blackburnit is just parameters passing, I think it is ok20:20
blackburnnot the main readability problem :)20:20
blackburne.g. DSYEVR routine have 22 args and it is not that difficult to read20:21
blackburnfor me at least20:21
@sonney2kblackburn, if there is a bug in creating this structure there it is very difficult to figure out which arg is the right one20:21
@sonney2kI consider any function with more than a handful of args 'broken by design'20:21
blackburnsonney2k: are you suggesting to revert the commit above? :)20:22
blackburnI spent a hour doing that :D20:23
@sonney2kI am suggesting to add the .x annotation20:23
@sonney2kotherwise reverting is better yes20:23
blackburnsonney2k: it is supported only by C99 standard20:23
@sonney2kbefore I had a chance to understand it - now I don't20:23
blackburnsonney2k: I will send a bomb to berlin :)20:25
@sonney2kwe have enough terrorist :)20:25
blackburnspent too much time on that20:25
@sonney2kone more doesnt' make a difference20:26
blackburnterrorist? do you have any?20:26
@sonney2kwhy not use the C99 syntax?20:26
blackburnsonney2k: produce warnings20:26
@sonney2kargh20:28
@sonney2kwhy is C99 stuff not in C++ ?!!20:29
blackburnsonney2k: that is the problem, yeah20:29
@sonney2kthen we cannot use it :(20:30
@sonney2khmmhh, maybe we should rethink the whole threading strategy20:30
@sonney2kI mean wouldn't it be possible to create some class for this business20:31
@sonney2k(templated I guess)20:31
@sonney2kand then add the usual functionality in there like the start/end things20:31
blackburnsonney2k: well yes, all of them contain some index start,step,stop20:31
@sonney2kI mean we have this appearing 20 or more times now20:32
@sonney2kit is getting #W$$!?!#?@20:32
blackburnsonney2k: a LOT of work20:33
@sonney2kalso we always solve the start the threads / join the threads / deal with error handling20:33
@sonney2kbut wisely spent20:33
@sonney2kmy suggestion would be to have start, end always in there and some templated thing called user_data or so20:35
@sonney2kthat contains the struct with required metadata for the task20:35
blackburnsonney2k: how to revert commit?20:36
@sonney2kgit revert <id>20:36
@sonney2kthen push again20:37
CIA-3shogun: Sergey Lisitsyn master * r15ebd97 / (7 files):20:38
CIA-3shogun: Revert "Improved multithreading codestyle, fixes some structure"20:38
CIA-3shogun: This reverts commit 51eeb3d7aeaebeb41297d6db822a073d03679e6b. - http://git.io/-pXtNQ20:38
blackburnsonney2k: yes, it could be one more bug-generating thing20:38
blackburnI would better stay it as it was20:38
@sonney2kblackburn, I guess you should open a trac ticket for this20:39
blackburnsonney2k: for what?20:39
@sonney2kfor this threading class20:39
@sonney2kI guess it will need a couple of iterations to be worth anything20:42
blackburn,20:43
blackburn?20:43
@sonney2kif we have to write 50 LoC to use that class it is not really an improvement20:43
@sonney2kalright - I will torture myself today with installing windows 7 again20:44
@sonney2kI hope I manage to do this somehow20:45
@sonney2kI never ever considered that this could be impossible when I have 2 harddrives...20:45
blackburnhah20:45
blackburnit would be impossible to install win7 to the second partition20:46
blackburnI Imagine what is the problems with multiple anything20:46
@sonney2kblackburn, well osx installed w/o problems to any harddrive20:47
@sonney2keven external usb drives20:47
@sonney2kand linux same thing20:47
blackburnno way to install win7 to flash drive :D20:47
@sonney2kthe problem is that I have to open up my notebook20:48
@sonney2kremove one harddisk20:48
@sonney2kand then install on the other20:48
@sonney2kand then pray that it works when I connect both of them20:48
blackburnsonney2k: does you notebook have two slots?20:49
@sonney2kblackburn, I removed the dvd drive20:49
blackburncrazy20:49
@sonney2kand connected a sata drive there20:49
@sonney2kwell when you have kids you suddenly have too many photos/movies20:49
@sonney2kand of course I was hoping to get shogun to run on cygwin / osx20:50
CIA-3shogun: Sergey Lisitsyn master * r9698ed3 / (4 files): Some necessary improvements from the reverted commit - http://git.io/eKzzgQ20:50
blackburnhah20:50
@sonney2kalright - if I won't be online in a few hrs - windows 7 killed my linux install20:52
@sonney2kcu20:52
blackburnokay see you20:53
-!- Guest56770 [~sonne@f053040237.adsl.alicedsl.de] has joined #shogun21:16
-!- Guest56770 [~sonne@f053040237.adsl.alicedsl.de] has quit [Ping timeout: 248 seconds]22:33
-!- blackburn [~blackburn@188.168.14.11] has quit [Ping timeout: 252 seconds]22:43
-!- blackburn [~blackburn@188.168.5.143] has joined #shogun22:45
--- Log closed Mon Oct 03 00:00:57 2011