--- Log opened Wed Nov 21 00:00:17 2012 | ||
wiking | ok | 00:01 |
---|---|---|
wiking | so it seems i can rebase now | 00:01 |
wiking | so what do i use instead of sg_realloc? | 00:05 |
@sonney2k | SG_REALLOC /SG_MALLOC / SG_CALLOC everything as usual | 00:05 |
wiking | structure/CCSOSVM.cpp:294:7: error: use of undeclared identifier 'SG_REALLOC' | 00:06 |
wiking | after rebase | 00:06 |
wiking | structure/CCSOSVM.cpp:298:50: error: too few arguments provided to function-like macro invocation | 00:06 |
blackburn | wiking: it was working before? | 00:07 |
wiking | yes | 00:07 |
wiking | it was | 00:07 |
blackburn | baad | 00:07 |
wiking | before rebase to the memory magix | 00:08 |
blackburn | :D | 00:08 |
wiking | so what the fuck | 00:08 |
blackburn | can yuo show these 294 and 298? | 00:08 |
wiking | https://github.com/shogun-toolbox/shogun/pull/823/files#L0R307 | 00:09 |
wiking | https://github.com/shogun-toolbox/shogun/pull/823/files#L0R311 | 00:09 |
blackburn | wiking: you should provide old size | 00:10 |
wiking | LOLFACE?! | 00:10 |
blackburn | type,pointer,old_len,len | 00:10 |
wiking | comooone?! | 00:10 |
wiking | why? | 00:10 |
wiking | have any of you seen man realloc ? :D | 00:10 |
wiking | i mean sorry but why would u need the oldsize? | 00:11 |
blackburn | because there is not realloc for new/delete | 00:11 |
blackburn | :D | 00:11 |
@sonney2k | wiking, that is new[ | 00:11 |
@sonney2k | ] | 00:11 |
@sonney2k | delete[] | 00:11 |
@sonney2k | and copy | 00:11 |
wiking | well realloc shouldnt behave like that | 00:11 |
@sonney2k | and no way to figure out how much to copy underneath | 00:11 |
@sonney2k | wiking, there is no realloc for new[] | 00:11 |
blackburn | it is not realloc in case of sgvectors | 00:11 |
wiking | my 2 cents | 00:11 |
@sonney2k | so that is why we need that | 00:11 |
wiking | fuck me in da face | 00:12 |
wiking | i mean if we want memory managment | 00:12 |
blackburn | see better solution? | 00:13 |
wiking | why u just don't take the fucking memory pool from kernel.org | 00:13 |
wiking | even better | 00:13 |
blackburn | how does it solve your problem? | 00:13 |
wiking | it was stupid of me to rebase | 00:13 |
blackburn | you don't know old size? | 00:14 |
wiking | well | 00:14 |
blackburn | wiking: put 0 here if you don't know | 00:14 |
wiking | yeah i have to maintain | 00:14 |
blackburn | it is not referenced anyway | 00:14 |
wiking | a new variable | 00:14 |
wiking | that like fucking tracks | 00:14 |
wiking | each fucking REALLOC's | 00:14 |
wiking | state | 00:14 |
wiking | fuckthishit | 00:15 |
blackburn | can't say adding zero is so much issue | 00:15 |
wiking | no but if i want to use it right | 00:15 |
wiking | that's what i should do | 00:16 |
blackburn | ? | 00:16 |
blackburn | it is used only if you realloc array of vectors | 00:16 |
wiking | http://www.boost.org/doc/libs/1_47_0/libs/pool/doc/index.html | 00:16 |
blackburn | wiking: so? | 00:17 |
wiking | i have 1000+ line of patch here | 00:19 |
wiking | and now trying to merge | 00:19 |
blackburn | so you want to change everything to use pool with one more 1000+ line patch? :D | 00:19 |
wiking | no i have some other patch | 00:20 |
wiking | and now i've rebased | 00:20 |
wiking | and now i'm broken | 00:20 |
wiking | so i have to merge now | 00:20 |
wiking | so that i can do anything | 00:20 |
wiking | like continue working | 00:20 |
wiking | libbm* shit has been minorly refactored | 00:21 |
wiking | but i have a feeling that that code should be rewriten in c++ | 00:22 |
blackburn | which code? | 00:22 |
wiking | libbm* | 00:23 |
wiking | oh yeah | 00:23 |
wiking | code is breaking | 00:23 |
blackburn | well I agree it could be better to write in C++ everywhere | 00:24 |
wiking | well there's so much code redundancy in that one | 00:24 |
blackburn | but that's legacy - do you want to write and test it once again | 00:24 |
wiking | well i've clean it up | 00:24 |
wiking | as much as i can | 00:25 |
blackburn | does it make use of vector operations? | 00:25 |
wiking | got like /2 speedup | 00:25 |
blackburn | because of eigen used? | 00:25 |
wiking | so it's like 2* faster... | 00:25 |
blackburn | for dot | 00:25 |
wiking | well eigen and cblas | 00:25 |
wiking | within each iteration it does at least 5 dots | 00:26 |
blackburn | we should think about alignment actually | 00:26 |
wiking | and another 3 of vec1_plus_alpha_vec2 stuff... | 00:26 |
wiking | but then again | 00:26 |
blackburn | feature vectors should be aligned | 00:26 |
wiking | sgvector.h has to be refactored | 00:26 |
wiking | i fucking hate to call static functions for a dot product | 00:26 |
wiking | well alignement | 00:28 |
wiking | is os/arch dependent | 00:28 |
blackburn | no | 00:28 |
wiking | so some cases u are lucky | 00:28 |
wiking | that the mem is alloced | 00:28 |
blackburn | in case of 3d vectors | 00:28 |
blackburn | we are always unlucky | 00:28 |
blackburn | I mean for eigen | 00:29 |
blackburn | I don't care about cblas as it as old as sonney2k :D | 00:29 |
wiking | :D | 00:31 |
blackburn | I didn't suggest that guy eigen3 | 00:32 |
blackburn | to not speed up nuclear weapons development | 00:32 |
blackburn | :D | 00:32 |
blackburn | oh I should stop joking about that | 00:32 |
blackburn | wiking: where have you seen that distCum? | 00:36 |
wiking | ahahahah | 00:37 |
wiking | of course a matlab code | 00:38 |
wiking | academics ;) | 00:38 |
blackburn | what does it mean? | 00:38 |
blackburn | distance okay | 00:38 |
blackburn | but what cum stands for in code? | 00:38 |
wiking | cumulative distance | 00:38 |
wiking | ;) | 00:38 |
blackburn | lol | 00:38 |
blackburn | wiking: I received math_anal.zip once | 00:38 |
wiking | dist-cum | 00:38 |
wiking | :D | 00:39 |
wiking | that's another great one | 00:39 |
blackburn | that was task for sth like 'mathematical analysis' | 00:39 |
wiking | calculs ;) | 00:39 |
wiking | calculus | 00:39 |
wiking | yeah | 00:39 |
blackburn | calculus right | 00:39 |
wiking | math anal | 00:39 |
wiking | :> | 00:39 |
blackburn | wiking: just today my neighbor gsomix showed me their teacher sent zad.zip | 00:40 |
blackburn | must be 'zadanie' - task/problem | 00:40 |
blackburn | but zad is effectively ass you know probably | 00:41 |
-!- zxtx [~zv@rrcs-24-213-167-69.nyc.biz.rr.com] has quit [Read error: Operation timed out] | 00:41 | |
wiking | :)))))) | 00:41 |
shogun-buildbot | build #149 of nightly_all is complete: Failure [failed compile] Build details are at http://www.shogun-toolbox.org/buildbot/builders/nightly_all/builds/149 | 03:00 |
shogun-buildbot | build #178 of nightly_default is complete: Success [build successful] Build details are at http://www.shogun-toolbox.org/buildbot/builders/nightly_default/builds/178 | 03:50 |
blackburn | wiking: http://metaoptimize.com/qa/questions/11465/should-cblas-triangular-matrices-be-full | 08:05 |
@sonney2k | wiking, did you stop crying? | 08:22 |
-shogungit:#shogun- [shogun] sonney2k pushed 1 new commit to master: https://github.com/shogun-toolbox/shogun/commit/0a5917168a713a2053ec5d72088cd001872a615d | 08:37 | |
-shogungit:#shogun- shogun/master 0a59171 Soeren Sonnenburg: include lib/memory.h in common.h again since SGVector forward declaration works ok now | 08:37 | |
sonne|work | wiking: b'cause you only have to change one line which is SG_REALLOC for this stuff to work | 09:11 |
sonne|work | shogun-buildbot: force rebuild nightly_all | 09:11 |
shogun-buildbot | try 'force build [--branch=BRANCH] [--revision=REVISION] [--props=PROP1=VAL1,PROP2=VAL2...] <WHICH> <REASON>' | 09:11 |
sonne|work | shogun-buildbot: force build nightly_all | 09:11 |
shogun-buildbot | build forced [ETA 28m17s] | 09:11 |
shogun-buildbot | I'll give a shout when the build finishes | 09:11 |
-!- blackburn [~blackburn@37.61.180.47] has quit [Quit: Leaving.] | 09:27 | |
shogun-buildbot | build #150 of nightly_all is complete: Success [build successful] Build details are at http://www.shogun-toolbox.org/buildbot/builders/nightly_all/builds/150 | 09:38 |
-!- blackburn [5bdfb203@gateway/web/freenode/ip.91.223.178.3] has joined #shogun | 11:03 | |
-!- zxtx [~zv@ip-64-134-65-129.public.wayport.net] has joined #shogun | 12:43 | |
-!- zxtx [~zv@ip-64-134-65-129.public.wayport.net] has quit [Ping timeout: 248 seconds] | 12:50 | |
-!- zxtx [~zv@ip-64-134-65-129.public.wayport.net] has joined #shogun | 13:07 | |
-shogungit:#shogun- [shogun] vigsterkr opened pull request #831: Add template specialization for SGVector's arg_max and max (master...master) https://github.com/shogun-toolbox/shogun/pull/831 | 13:08 | |
wiking | a little bit of eyecandy first ;D | 13:08 |
-shogungit:#shogun- [shogun] sonney2k pushed 2 new commits to master: https://github.com/shogun-toolbox/shogun/compare/0a5917168a71...ba1e9afb7ddc | 13:21 | |
-shogungit:#shogun- shogun/master 1f02914 Viktor Gal: Add template specialization for SGVector's arg_max and max... | 13:21 | |
-shogungit:#shogun- shogun/master ba1e9af Soeren Sonnenburg: Merge pull request #831 from vigsterkr/master... | 13:21 | |
sonne|work | wiking: heh | 13:21 |
sonne|work | wiking: no idea how the discussion yesterday night went on | 13:22 |
sonne|work | but what are your complaints exactly? | 13:22 |
blackburn | sonne|work: he is just like me :D | 13:22 |
sonne|work | blackburn: which means? | 13:22 |
wiking | that realloc api | 13:22 |
wiking | nonstandard | 13:22 |
blackburn | sonne|work: marvin | 13:22 |
sonne|work | I see | 13:23 |
wiking | ah shit | 13:23 |
sonne|work | wiking: yes I know. I couldn't come up with any standard solution. If you have an idea speak up | 13:23 |
wiking | i forgot to change sparsematrix | 13:23 |
sonne|work | wiking: fwiw I don't like it either but whenever we do realloc in shogun we know the old size so this is the least intrusive change | 13:24 |
wiking | mmm | 13:27 |
sonne|work | the only other option I see is to use new SGVector<int>[10]() and delete[] throughout the code | 13:28 |
sonne|work | etc | 13:28 |
blackburn | or restrict realloc for sgvectors | 13:29 |
sonne|work | blackburn: yeah but then we cannot use DynArray for SGvector | 13:29 |
blackburn | bad | 13:29 |
sonne|work | blackburn: btw you said std::vector works for cases that need inplace constructors? | 13:30 |
blackburn | sonne|work: I am curious would http://ustl.sourceforge.net/ work for you? | 13:30 |
blackburn | sonne|work: vector calls constructors yes | 13:30 |
blackburn | it has no chance to fail on that no matter what class it is | 13:31 |
sonne|work | blackburn: any idea how it know when to call constructors? | 13:31 |
blackburn | sonne|work: what do you mean 'when'? | 13:31 |
sonne|work | I mean vector of double works too right? | 13:31 |
sonne|work | well it needs to know that it can call the constructor | 13:31 |
blackburn | yes why not | 13:31 |
blackburn | are you sure new double[]() doesn't work? | 13:32 |
blackburn | I think it does actually | 13:32 |
sonne|work | I didn't try but then I wonder why plain new[] exists... | 13:34 |
wiking | ? | 13:36 |
blackburn | sonne|work: take a look at stl code :) | 13:36 |
wiking | blackburn: new double[] should work | 13:37 |
blackburn | wiking: new double[30]();? | 13:37 |
wiking | mmm | 13:37 |
sonne|work | blackburn: well it compiles... | 13:38 |
blackburn | hahaha youtube is banned in my country OKAY | 13:39 |
wiking | blackburn: lol? | 13:39 |
sonne|work | what? | 13:39 |
sonne|work | what happened? | 13:39 |
blackburn | youtube is illegal :D | 13:40 |
blackburn | lol | 13:40 |
wiking | blackburn: can u open it still? | 13:40 |
blackburn | yes | 13:40 |
wiking | so what? | 13:40 |
blackburn | but won't be able soon I think | 13:40 |
wiking | if they catch u watching it, then they'll come and fuck you up? :) | 13:40 |
blackburn | they introduced a list of sites that are denied | 13:40 |
wiking | ah yeah | 13:40 |
wiking | the blacklist | 13:40 |
blackburn | so just a few hours ago they put youtube there | 13:40 |
wiking | i mean hey | 13:41 |
blackburn | and now ISPs will deny it | 13:41 |
wiking | you don't wanna watch that shit anyways | 13:41 |
wiking | :))) | 13:41 |
blackburn | haha right | 13:41 |
wiking | maybe there's THAT kind of content there :P | 13:41 |
blackburn | wiking: they tried to ban it before because of 'innocence of muslims' | 13:42 |
blackburn | fucking muslims they are trouble makers all around the world | 13:42 |
wiking | another fixfuck for shogun | 13:43 |
-shogungit:#shogun- [shogun] vigsterkr opened pull request #832: Fix SGSparseMatrix ctor to comply with SGMatrix ctor (master...master) https://github.com/shogun-toolbox/shogun/pull/832 | 13:43 | |
wiking | blackburn: what should new double[]() be? | 13:43 |
blackburn | wiking: call ctor of double | 13:43 |
sonne|work | wiking: double[] | 13:44 |
-shogungit:#shogun- [shogun] lisitsyn pushed 2 new commits to master: https://github.com/shogun-toolbox/shogun/compare/ba1e9afb7ddc...810c9e5b7e6e | 13:44 | |
-shogungit:#shogun- shogun/master c7f0a72 Viktor Gal: Fix SGSparseMatrix ctor to comply with SGMatrix ctor... | 13:44 | |
-shogungit:#shogun- shogun/master 810c9e5 Sergey Lisitsyn: Merge pull request #832 from vigsterkr/master... | 13:44 | |
sonne|work | maybe it inits with 0's ? | 13:44 |
blackburn | sonne|work: yes | 13:44 |
sonne|work | wiking: did you convert all places that call SGSparseMatrix? | 13:45 |
sonne|work | I mean otherwise stuff is transposed... | 13:45 |
blackburn | sonne|work: wait but doesn't new Class[] call constructors? | 13:46 |
sonne|work | blackburn: no | 13:46 |
wiking | sonne|work: yeah i've tried to cover all of it as you can see in the patch | 13:46 |
sonne|work | or? | 13:46 |
sonne|work | wiking: k | 13:46 |
blackburn | sonne|work: so you mean new Class[30] is unsafe later? I am unsure about that | 13:46 |
sonne|work | http://stackoverflow.com/questions/620137/do-the-parentheses-after-the-type-name-make-a-difference-with-new | 13:47 |
wiking | blackburn: new should call the default ctor | 13:47 |
sonne|work | crazy crazy crazy | 13:47 |
sonne|work | didn't know one can write new Class; | 13:47 |
wiking | yes | 13:47 |
blackburn | sonne|work: yes sure one can | 13:47 |
blackburn | but I was unsure about new[]() and new[] | 13:47 |
blackburn | so we just can use new[] and it will work nice | 13:48 |
sonne|work | blackburn: yeah but it has a different meaning | 13:48 |
blackburn | what's different? | 13:48 |
sonne|work | look at the above link | 13:49 |
wiking | sonne|work: well depends ;) | 13:49 |
wiking | i mean blackburn :) | 13:49 |
wiking | it depends | 13:49 |
blackburn | sonne|work: no, new[] would work in all our cases I think | 13:49 |
wiking | i mean if u pass a class | 13:50 |
sonne|work | blackburn: no I think it won't work like in the above link stuff will be uninited | 13:50 |
wiking | then it should work as all classes have by default a ctor and even a copy ctor (as c++ automagically defines one) if you dont enforce it to be private | 13:50 |
blackburn | sonne|work: things like int? | 13:50 |
sonne|work | uninited | 13:50 |
blackburn | sonne|work: okay actually new[size]() is ok for me | 13:51 |
blackburn | it is rather universal | 13:51 |
blackburn | always works no matter simple type or class is used | 13:51 |
wiking | we always have default ctor for every class anyways for serialization | 13:51 |
sonne|work | blackburn: yeah for 'higher' types that makes sense | 13:52 |
blackburn | wiking: sonne|work it took 2 hr to recognize they are doing something wrong and youtube is not banned anymore :D | 14:14 |
wiking | :> | 14:17 |
wiking | that was fast actually | 14:17 |
blackburn | great soviet firewall | 14:33 |
-!- zxtx [~zv@ip-64-134-65-129.public.wayport.net] has quit [Ping timeout: 248 seconds] | 15:39 | |
-!- zxtx [~zv@ip-64-134-65-129.public.wayport.net] has joined #shogun | 15:42 | |
-!- zxtx [~zv@ip-64-134-65-129.public.wayport.net] has quit [Ping timeout: 255 seconds] | 15:48 | |
-!- blackburn [5bdfb203@gateway/web/freenode/ip.91.223.178.3] has quit [Quit: Page closed] | 15:49 | |
-!- zxtx [~zv@ip-64-134-65-129.public.wayport.net] has joined #shogun | 15:50 | |
-!- zxtx [~zv@ip-64-134-65-129.public.wayport.net] has quit [Ping timeout: 246 seconds] | 17:01 | |
-!- blackburn [~blackburn@83.234.169.54] has joined #shogun | 17:02 | |
@sonney2k | muaahhh | 17:41 |
-shogungit:#shogun- [shogun] sonney2k pushed 2 new commits to master: https://github.com/shogun-toolbox/shogun/compare/810c9e5b7e6e...217af0036e06 | 18:17 | |
-shogungit:#shogun- shogun/master dfc7d7d Soeren Sonnenburg: move function definitions to .cpp file | 18:17 | |
-shogungit:#shogun- shogun/master 217af00 Soeren Sonnenburg: define index_t in lib/common.h like all other datatypes | 18:17 | |
@sonney2k | blackburn, now see what you have done - you could have suggested eigen3 | 18:19 |
blackburn | sonney2k: hahahaha | 18:19 |
-!- shogun-buildbot [~shogun-bu@7nn.de] has quit [Remote host closed the connection] | 18:20 | |
--- Log opened Wed Nov 21 18:21:32 2012 | ||
-!- shogun-toolbox [~shogun@7nn.de] has joined #shogun | 18:21 | |
-!- Irssi: #shogun: Total of 5 nicks [0 ops, 0 halfops, 0 voices, 5 normal] | 18:21 | |
-!- Irssi: Join to #shogun was synced in 6 secs | 18:21 | |
-!- shogun-buildbot [~shogun-bu@7nn.de] has joined #shogun | 18:21 | |
-!- sonney2k [~shogun@7nn.de] has joined #shogun | 18:21 | |
-!- mode/#shogun [+o sonney2k] by ChanServ | 18:21 | |
-!- shogun-buildbot [~shogun-bu@7nn.de] has quit [Remote host closed the connection] | 18:27 | |
--- Log opened Wed Nov 21 18:28:27 2012 | ||
-!- shogun-toolbox [~shogun@7nn.de] has joined #shogun | 18:28 | |
-!- Irssi: #shogun: Total of 5 nicks [0 ops, 0 halfops, 0 voices, 5 normal] | 18:28 | |
-!- Irssi: Join to #shogun was synced in 7 secs | 18:28 | |
-!- shogun-buildbot [~shogun-bu@7nn.de] has joined #shogun | 18:28 | |
-!- sonney2k [~shogun@7nn.de] has joined #shogun | 18:28 | |
-!- mode/#shogun [+o sonney2k] by ChanServ | 18:29 | |
-!- zxtx [~zv@ool-44c1360a.dyn.optonline.net] has joined #shogun | 19:59 | |
@sonney2k | wiking, around? | 20:51 |
@sonney2k | wiking, I wonder whether the sparse dot product now assumes sorted indices or not. | 20:51 |
-shogungit:#shogun- [shogun] sonney2k pushed 1 new commit to master: https://github.com/shogun-toolbox/shogun/commit/7528d0e12453396aa3d47ad08bdde6446c70af70 | 21:06 | |
-shogungit:#shogun- shogun/master 7528d0e Soeren Sonnenburg: put impl. of sparse vector / matrix functions into .cpp | 21:06 | |
wiking | sonney2k: it does | 21:26 |
wiking | even the comment says so | 21:27 |
wiking | that we suppose that the indicies are ordered in non descending manner | 21:27 |
shogun-buildbot | build #179 of nightly_default is complete: Failure [failed tutorial] Build details are at http://www.shogun-toolbox.org/buildbot/builders/nightly_default/builds/179 | 21:36 |
--- Log closed Thu Nov 22 00:00:15 2012 |
Generated by irclog2html.py 2.10.0 by Marius Gedminas - find it at mg.pov.lt!