IRC logs of #shogun for Wednesday, 2011-06-29

--- Log opened Wed Jun 29 00:00:29 2011
-!- f-x [~user@117.192.197.13] has joined #shogun00:02
blackburnwhoaa00:03
@sonney2kf-x, around?00:17
@sonney2kif so what do you think - did my comment make sense00:17
@sonney2k?00:17
f-xsonney2k: yeah, present00:19
f-xsonney2k: actually just commented on github on that very comment00:19
f-xit does make sense, but what would go into say a CSimpleStream class?00:20
f-xonly the pure virtual get_vector() function, i'm thinking00:20
@sonney2kyes the interface00:21
f-xokay, so at least i understood correctly00:22
@sonney2kso you could rely on these functions being available and don't have to do extra checks00:22
f-xthat's a nice thing to have00:22
f-xand can i use templating?00:22
f-xfor the CSimpleStream class?00:22
@sonney2kf-x, you run into the inheritance problem then00:23
@sonney2kso no00:23
@sonney2kyou could however use overloading if that doesn't break00:23
f-xsonney2k: are you sure about that inheritance problem?00:24
@sonney2kyes00:24
blackburnhmmm how to shorten Distance-based Dimension Reduction Preprocessor00:24
blackburnhaha base could be class NLDR (nonlinear dimension reduction) and derived is dbdrplandmarkmds00:25
blackburnsonney2k: you should like it - deutsch have looong words too :)00:26
@sonney2kblackburn, I have no idea00:26
blackburnsonney2k: just joking ;)00:26
blackburnsonney2k: is it a complete C.elegans genome in our data?00:27
blackburnhow bad we can't run this dna :(00:28
blackburnI would want to see a huge worm with legs or so00:28
f-xsonney2k: how about templating just the derived CSimpleStreamFromSimpleFeatures and CSimpleAsciiStream?00:30
f-xi think that could work00:30
@sonney2kf-x, I don't understand what exactly you want to do00:35
@sonney2kthen you have no read baseclass - right?00:36
@sonney2kerr SimpleStream00:36
f-xsonney2k: no, all that is still there00:36
f-xit changes one thing00:36
@sonney2kf-x, but no interfaces or?00:36
f-xin the StreamingSimpleFeatures class00:37
f-xset_vector_reader just makes the reader a generic "get_vector" function00:37
f-xinstead of get_bool_vector or get_real_vector etc00:37
@sonney2kf-x, for which type?00:37
@sonney2kor for all types?00:37
f-xfor all types... it is done now through a #define00:38
f-xin StreamingSimpleFeatures00:38
f-xget_vector could be a specialized template function for each type T00:38
f-xand then StreamingSimpleFeatures would just use the specialized get_vector for that template type00:39
@sonney2kf-x, wait get_vector in StreamingSimpleFeatures cannot be templated - but in the derived classes it could but then I don't think you benefit00:39
f-xsonney2k: sorry, not that get_vector00:40
f-xi'm confusing you00:40
f-xi'm talking about the functions which read a vector from a file00:41
@sonney2kahh yes but still problem remains (despite the name)00:41
f-xthey are called get_word_vector, get_int_vector, etc..00:41
@sonney2kcurrently - yes00:41
@sonney2kso what would you do now instead?00:42
@sonney2kwrite one templated function for that00:42
f-xcall it read_vector() or something which doesn't clash00:42
f-xyes00:42
@sonney2kand then call it in the get_*_vector?00:42
f-xwait00:42
f-xit's becoming roundabout00:43
f-x#define SET_VECTOR_READER(sg_type, sg_function)\00:43
f-xtemplate <> void CStreamingSimpleFeatures<sg_type>::set_vector_reader() \00:43
f-x{\00:43
f-xparser.set_read_vector(&CStreamingFile::sg_function);\00:43
f-x}00:43
f-x 00:43
f-xSET_VECTOR_READER(bool, get_bool_vector);00:43
@sonney2kI mean I would wish one could have a templated get_vector function in the base class - but in the derived classes you still would need to overload each templated variant00:43
f-xSET_VECTOR_READER(char, get_char_vector);00:43
f-xSET_VECTOR_READER(int8_t, get_int8_vector);00:43
f-x 00:43
f-xthat's part of the code which sets the read functions in StreamingSimpleFeatures00:43
f-xsonney2k: yes, you're right00:44
f-xsome implementation would have to be given for each T00:44
@sonney2kyeah so I think you cannot do much - I think the most clean thing you could do is to use a templated function in the derived class00:44
@sonney2kand have that called from the get_*_vector() functions00:44
@sonney2kthen at least the function code is not hidden in the macro00:45
blackburnbase/class_list.cpp: In function ‘shogun::CSGObject* __new_CLandmarkEIsomap(shogun::EPrimitiveType)’:00:45
blackburnbase/class_list.cpp:287: error: expected type-specifier before ‘CLandmarkEIsomap’00:45
blackburnsonney2k: any ideas?00:45
@sonney2kblackburn, did you do the usual distclean?00:45
blackburnwill try again but seems it won't help00:46
@sonney2kf-x, btw - the interface for streamingfeatures is much better now00:46
* sonney2k becomes *very* sleepy00:46
f-xsonney2k: thanks.. i've begun to like it too :)00:47
f-xi'll try to integrate the labels into the SimpleStreamFromSimpleFeatures-like classes soon00:47
@sonney2kf-x, so what do you think about the splitting up of StreamingFile?00:48
f-xsonney2k: your idea is sensible, i agree00:48
blackburnsonney2k: haven't helped00:48
f-xthere wasn't much to gain by using the current scheme anyway00:49
@sonney2kf-x, yeah...00:49
f-xbut our names are becoming more java-ish day by day00:49
@sonney2kblackburn, hmmhh I suspect you forgot an include somewhere00:50
@sonney2kf-x, how so?00:50
f-xCStreamingFileFromSimpleFeatures00:50
f-xthat looks like a java trademark00:50
@sonney2kf-x, well I don't have an idea how to shorten this00:50
f-xsonney2k: me neither.. just saying00:51
@sonney2kit is only meaningful this way00:51
@sonney2kok00:51
blackburnsonney2k: CEIsomap is defined in EIsomap.h and CLandmarkEIsomap here too00:52
@sonney2kf-x, so please do this split up then - I think the code will be even more readable then :)00:53
blackburnI have similar strange problem while working on ContingencyTableEvaluation00:53
bettyboohe00:53
f-xsonney2k: sure.. expect it real soon00:53
@sonney2kblackburn, yeah but you forgot an include there too00:53
@sonney2k(at least that is what I remember)00:53
@sonney2kblackburn, btw what is EIsomap?00:54
@sonney2kf-x thanks!00:54
f-xsonney2k: btw will we be using CLabels for StreamingFeatures too?00:54
blackburnsonney2k: E-Isomap00:54
blackburnwell there are two isomap 'ways'00:54
blackburnto connect edges with weights less than epsilon - E-Isomap00:55
blackburnor to connect just k neighors - K-Isomap00:55
@sonney2kf-x, well return just a float64_t - for simplestreamingfeatures you should expect a simplefeature object and labels (optional)00:55
@sonney2kblackburn, I see00:55
f-xsonney2k: ok.. so i won't encapsulate everything in a CLabels object yet00:56
@sonney2kf-x, it doesn't really make sense - you only ever return a scalar (at least now)00:58
@sonney2kf-x, and btw. for StreamingAsciiFile you could specify the CAsciFile in constructor and use CAsciiFile's functions00:59
blackburnI should think before coding01:00
f-xsonney2k: for reading the file? but CAsciiFile reads the complete file, right?01:00
@sonney2kf-x, you can use its read_line function01:01
@sonney2k(or whatever it is called)01:01
f-xsonney2k: it's being used in StreamingAsciiFile through CAsciiFile::getline right now, though it could be done in a better way01:03
f-xi'll modify it a bit01:03
@sonney2kyeah meant that function01:03
@sonney2kok I have to sleep now01:03
@sonney2kcu all!01:04
f-xsee you!01:04
blackburnsee ya01:06
-!- f-x [~user@117.192.197.13] has quit [Remote host closed the connection]01:41
CIA-32shogun: Sergey Lisitsyn master * r398048d / (11 files in 2 dirs): Various fixes and changes for Isomap - http://bit.ly/mKx7Dp01:52
CIA-32shogun: Sergey Lisitsyn master * r427a6c6 / (11 files in 2 dirs): Various fixes and changes for Isomap - http://bit.ly/kKNzAV01:54
CIA-32shogun: Sergey Lisitsyn master * rb8eca36 / :01:57
CIA-32shogun: Merge branch 'master' of ssh://shogun-toolbox.org/shogun01:57
CIA-32shogun: Conflicts:01:57
CIA-32shogun:  src/modular/Preprocessor.i - http://bit.ly/jj54fw01:57
blackburnah sorry I forgot to amend commit before push01:59
-!- blackburn [~blackburn@31.28.40.202] has quit [Quit: Leaving.]02:00
-!- alesis-novik [~alesis@188.74.87.206] has quit [Ping timeout: 252 seconds]02:33
-!- alesis-novik [~alesis@vpn2-244.vpn.net.ed.ac.uk] has joined #shogun02:45
-!- in3xes [~in3xes@210.212.58.111] has quit [Ping timeout: 240 seconds]06:28
-!- in3xes [~in3xes@180.149.49.227] has joined #shogun06:41
-!- in3xes [~in3xes@180.149.49.227] has quit [Ping timeout: 240 seconds]06:47
-!- in3xes [~in3xes@180.149.49.227] has joined #shogun06:57
-!- heiko [~heiko@main.uni-duisburg.de] has joined #shogun10:01
-!- in3xes_ [~in3xes@180.149.49.227] has joined #shogun10:19
-!- in3xes [~in3xes@180.149.49.227] has quit [Ping timeout: 240 seconds]10:22
-!- in3xes1 [~in3xes@180.149.49.227] has joined #shogun10:38
-!- in3xes_ [~in3xes@180.149.49.227] has quit [Ping timeout: 244 seconds]10:42
heikosonney2k are you there?10:50
heikoi think I found a bug in DynArray: the operator = has a problem if the array that is to be copied is much larger than the array that is copied into.10:51
heikowill try to fix it.10:51
-!- in3xes_ [~in3xes@180.149.49.227] has joined #shogun10:54
-!- in3xes1 [~in3xes@180.149.49.227] has quit [Ping timeout: 240 seconds]10:57
-!- in3xes_ [~in3xes@180.149.49.227] has quit [Ping timeout: 252 seconds]11:02
-!- blackburn [~blackburn@31.28.40.202] has joined #shogun11:22
CIA-32shogun: Soeren Sonnenburg master * r827f2b0 / (21 files in 6 dirs):11:36
CIA-32shogun: Merge pull request #153 from karlnapf/master11:36
CIA-32shogun: new subset implementation (+7 more commits...) - http://bit.ly/jqI4sV11:36
heikosonney2k are you there?11:58
blackburnheiko: I can merge your pull12:26
blackburn*if you want* :)12:27
heikohej blackburn :)12:37
heikoyes, if you think its ok12:37
heikoI am not sure whether soeren likes the SG_CALLOC stuff. The bugfix should be ok12:39
heikowill go for lunch now12:39
heikosee you :)12:39
blackburnheiko: ok 'll wait for Soeren :)12:43
blackburnenjoy your meal12:43
-!- heiko [~heiko@main.uni-duisburg.de] has quit [Ping timeout: 258 seconds]12:51
blackburnhmm I guess some changes caused really slow compilation13:18
-!- in3xes [~in3xes@180.149.49.227] has joined #shogun13:27
-!- in3xes_ [~in3xes@210.212.58.111] has joined #shogun13:36
-!- in3xes [~in3xes@180.149.49.227] has quit [Ping timeout: 276 seconds]13:39
-!- alesis-novik [~alesis@vpn2-244.vpn.net.ed.ac.uk] has quit [Ping timeout: 246 seconds]13:44
-!- heiko [~heiko@main.uni-duisburg.de] has joined #shogun13:52
-!- blackburn [~blackburn@31.28.40.202] has quit [Quit: Leaving.]13:56
-!- alesis-novik [~alesis@188.74.87.206] has joined #shogun13:59
-!- sploving1 [~sploving@2001:cc0:2020:2011:5eff:35ff:fe04:f091] has joined #shogun14:24
sploving1anyone here?14:30
sploving1./preprocessor/ClassicIsomap.h:50: error: ‘m_target_dim’ was not declared in this scope14:31
sploving1base/class_list.cpp: In function ‘shogun::CSGObject* __new_CLandmarkIsomap(shogun::EPrimitiveType)’:14:31
sploving1base/class_list.cpp:436: error: cannot convert ‘shogun::CLandmarkIsomap*’ to ‘shogun::CSGObject*’ in return14:31
sploving1base/class_list.cpp: In function ‘shogun::CSGObject* __new_CClassicIsomap(shogun::EPrimitiveType)’:14:31
sploving1base/class_list.cpp:439: error: cannot convert ‘shogun::CClassicIsomap*’ to ‘shogun::CSGObject*’ in return14:31
-!- in3xes [~in3xes@180.149.49.227] has joined #shogun14:34
sploving1I merged upstream and configure java_modular14:34
sploving1it shows the above error14:35
sploving1any one could help?14:35
heikohi sploving14:39
heikosorry, I do not have java configured14:39
heikocurrently struggeling with java14:39
heikoehm python_modula14:39
heikor14:39
-!- in3xes [~in3xes@180.149.49.227] has quit [Ping timeout: 260 seconds]14:41
sploving1I think maybe someone changes for Isomap14:42
-!- in3xes [~in3xes@180.149.49.227] has joined #shogun14:49
-!- in3xes__ [~in3xes@180.149.49.227] has joined #shogun14:53
-!- in3xes [~in3xes@180.149.49.227] has quit [Ping timeout: 246 seconds]14:57
-!- in3xes [~in3xes@180.149.49.227] has joined #shogun15:00
-!- in3xes__ [~in3xes@180.149.49.227] has quit [Ping timeout: 252 seconds]15:03
-!- in3xes__ [~in3xes@180.149.49.227] has joined #shogun15:04
-!- sploving1 [~sploving@2001:cc0:2020:2011:5eff:35ff:fe04:f091] has left #shogun []15:05
-!- in3xes [~in3xes@180.149.49.227] has quit [Ping timeout: 258 seconds]15:07
-!- in3xes__ [~in3xes@180.149.49.227] has quit [Ping timeout: 255 seconds]15:12
-!- in3xes__ [~in3xes@180.149.49.227] has joined #shogun15:25
-!- heiko [~heiko@main.uni-duisburg.de] has quit [Ping timeout: 258 seconds]15:28
-!- in3xes__ is now known as in3es15:32
-!- in3es is now known as in3xes15:32
-!- in3xes__ [~in3xes@180.149.49.227] has joined #shogun15:34
-!- in3xes [~in3xes@180.149.49.227] has quit [Ping timeout: 244 seconds]15:37
-!- in3xes1 [~in3xes@180.149.49.227] has joined #shogun15:37
-!- in3xes__ [~in3xes@180.149.49.227] has quit [Ping timeout: 252 seconds]15:39
-!- in3xes_ [~in3xes@210.212.58.111] has quit [Ping timeout: 276 seconds]15:41
-!- f-x [~user@117.192.219.76] has joined #shogun16:13
-!- in3xes_ [~in3xes@180.149.49.227] has joined #shogun16:15
-!- in3xes1 [~in3xes@180.149.49.227] has quit [Ping timeout: 240 seconds]16:19
-!- in3xes_ is now known as in3xes16:25
-!- heiko [~heiko@main.uni-duisburg.de] has joined #shogun17:11
-!- in3xes_ [~in3xes@180.149.49.227] has joined #shogun17:20
-!- in3xes [~in3xes@180.149.49.227] has quit [Ping timeout: 263 seconds]17:24
-!- in3xes_ is now known as in3xes17:30
-!- f-x [~user@117.192.219.76] has quit [Read error: Connection reset by peer]17:39
-!- in3xes_ [~in3xes@180.149.49.227] has joined #shogun17:39
-!- heiko [~heiko@main.uni-duisburg.de] has quit [Ping timeout: 258 seconds]17:42
-!- in3xes [~in3xes@180.149.49.227] has quit [Ping timeout: 240 seconds]17:43
-!- in3xes_ is now known as in3xes17:43
-!- in3xes_ [~in3xes@180.149.49.227] has joined #shogun17:50
-!- in3xes [~in3xes@180.149.49.227] has quit [Ping timeout: 240 seconds]17:54
-!- in3xes1 [~in3xes@59.163.196.121] has joined #shogun17:56
-!- in3xes_ [~in3xes@180.149.49.227] has quit [Ping timeout: 240 seconds]17:59
-!- f-x [~user@117.192.219.76] has joined #shogun18:06
-!- in3xes1 is now known as in3xes18:43
-!- blackburn [~blackburn@31.28.40.202] has joined #shogun19:26
blackburnoh I forgot some includes.. how it was working on my19:33
blackburnsonney2k: some problem with heiko's patch: it is not compiling now :)20:35
CIA-32shogun: Sergey Lisitsyn master * rc47b8ec / src/libshogun/preprocessor/Isomap.h : Fixed includes at Isomap.h - http://bit.ly/iYQux220:48
blackburneh... everything is ok.21:05
blackburnwtf21:05
-!- blackburn [~blackburn@31.28.40.202] has quit [Ping timeout: 255 seconds]21:12
@sonney2kall good now?21:18
CIA-32shogun: Soeren Sonnenburg master * re5b3696 / src/java_modular/swig_typemaps.i :21:27
CIA-32shogun: Merge pull request #155 from sploving/master21:27
CIA-32shogun: support SGVector for UJMP - http://bit.ly/mLWtUq21:27
-!- blackburn [~blackburn@31.28.40.202] has joined #shogun21:28
blackburnsonney2k: around?21:30
@sonney2kyes21:30
@sonney2kall good now?21:31
blackburnsonney2k: nope21:31
blackburnsonney2k: is it compiling on your machine?21:31
@sonney2k?21:31
@sonney2kcannot test right now21:31
blackburnreally strange errors21:31
blackburn../shogun/features/Features.h:282: error: ‘shogun::CSubset* shogun::CFeatures::m_subset’ is protected21:31
blackburn../shogun/features/StringFeatures.h:1685: error: within this context21:31
blackburn../shogun/features/Features.h: In member function ‘bool shogun::CStringFeatures<ST>::obtain_from_char_features(shogun::CStringFeatures<CT>*, int32_t, int32_t, int32_t, bool) [with CT = char, ST = int]’:21:31
@sonney2kwhat does not work?21:31
blackburnthe most strange thing about it - it fails on wrappers compilation21:32
@sonney2khmmh i cannot make sense of this21:33
blackburnsonney2k: I guess I know21:34
blackburnmay be subset.h should be imported?21:34
CIA-32shogun: Soeren Sonnenburg master * rb803c05 / (8 files in 4 dirs):21:36
CIA-32shogun: Merge pull request #154 from karlnapf/master21:36
CIA-32shogun: bug in DynArray - http://bit.ly/l7sCtm21:36
@sonney2kblackburn, in Features_includes.i21:37
@sonney2kand Features.i21:37
@sonney2kmakes sense21:37
@sonney2kand rename too21:37
blackburnsonney2k: I have added to stirng features, compiling now21:37
blackburnwill check in a min21:37
@sonney2kf-x, any news - are you working on the patch21:39
@sonney2k?21:39
blackburnnope21:39
f-xsonney2k: yeah... actually had some clarifications to make21:40
f-xwhat functions actually go into the SimpleStream interface?21:40
@sonney2kthe vector ones21:41
f-xget_int_vector(), get_real_vector() etc., right?21:42
f-xand get_*_vector_and_label()...?21:42
blackburnsonney2k: import haven't helped.21:44
@sonney2kblackburn, no import!21:45
@sonney2kinclude21:45
blackburninclude yes21:45
@sonney2kbaby crying...21:45
@sonney2kl8r21:45
blackburnnow should be fixed21:57
blackburnheiko did it indeed wrong21:58
blackburnit should not work21:58
blackburnthere was an access to protected field21:58
CIA-32shogun: Sergey Lisitsyn master * rcc04bff / src/libshogun/features/StringFeatures.h : Added FIXME and fixed StringFeatures compile bug - http://bit.ly/mkHgJE22:47
* blackburn is translating FAQ to russian hehe22:57
@sonney2kblackburn, I think heiko's code is right23:17
blackburnsonney2k: it is not compilable23:17
@sonney2km_subset is protected yes but it is defined in CFeatures and stringfeatures derives from that23:17
@sonney2kso it can access m_subset23:18
blackburnsonney2k: it is passed as parameter23:18
blackburnit is ok for g++ but swig fails on that23:18
blackburnsonney2k: not just m_subset but m_subset of StringFeatures passed to that method as parameter I mean23:19
@sonney2khmmhh I thought it doesn' t matter if this is a parameter23:19
blackburnsf->m_subset23:19
@sonney2kyes but I don't see what could be wrong here23:20
blackburnjust try to compile it23:20
blackburnI even re-cloned shogun :D23:20
blackburntest23:31
@sonney2kblackburn, I think this is because of the template thing23:32
@sonney2kI mean otherwise lines such as sf->m_subset->subset_idx_conversion(i); would also cause errors (sf is CStringFeatures<ST>* sf)23:33
blackburnsonney2k: don't know, the only thing I wonder - is it compiles on heiko's machine? :)23:33
bettyboonot bad!?!23:33
@sonney2kyeah that is strange23:34
blackburnfuck.. my K-Isomap is wrong. again!23:34
blackburnsonney2k: what would you do if you was needed to remove NOT k-nearest neighbor edges from distance matrix?23:37
blackburn-> infty them23:37
@sonney2kmakes sense23:38
blackburnI mean my very naive way to do that23:38
blackburnsort row(or column) indexes23:38
blackburnwith decreasing distance23:38
blackburnand just -> infty starting from k+123:39
blackburnthe most surprising thing about that - this will produce NONsymmetric matrix :(23:40
blackburnhmm may be just 'symmetrize' it..23:44
blackburnI really wonder I'm stucked on such a EASY thing :D23:49
--- Log closed Thu Jun 30 00:00:31 2011

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