IRC logs of #shogun for Wednesday, 2017-05-24

« Tuesday, 2017-05-23 Index Thursday, 2017-05-25 »
--- Log opened Wed May 24 00:00:43 2017
-!- witness_ [uid10044@gateway/web/irccloud.com/x-nowpvqgxvtbfunme] has quit [Quit: Connection closed for inactivity]01:04
-!- witness_ [uid10044@gateway/web/irccloud.com/x-hqpdwwbckniyzchm] has joined #shogun01:40
@sukeyPull Request #3819 "make guassian get rid of LAPACK"  synchronized by MikeLing - https://github.com/shogun-toolbox/shogun/pull/381903:55
@sukeyPull Request #3819 "make guassian get rid of LAPACK"  synchronized by MikeLing - https://github.com/shogun-toolbox/shogun/pull/381904:02
-!- mikeling|afk is now known as mikeling04:04
mikelingwiking: ping04:11
-!- witness_ [uid10044@gateway/web/irccloud.com/x-hqpdwwbckniyzchm] has quit [Quit: Connection closed for inactivity]04:14
mikelinglisitsyn: ping04:35
@wikingpong07:12
mikelingwiking: Hi, do you have time to review this  https://github.com/shogun-toolbox/shogun/pull/3819?07:51
@wikingyeah let's ahve a chat in 5 minutes07:51
@wiking*have07:51
mikelingsure07:51
@wikingjust a sec08:03
mikelingnp :)08:03
@wikingk here08:12
@wikingso i'm just checking something08:12
mikelingok08:12
@wikingso08:13
@wikingwe have  this ancient wrapper for some lapack functionallities08:13
@wiking./src/shogun/mathematics/lapack.h08:13
mikelingyep08:14
@wikingso i'm just wondering whether we should add something there08:16
@wikingwe could do a big08:16
@wiking#endif //HAVE_LAPACK08:16
@wiking#else08:17
@wikingand there add some functionalities08:17
@wikingbut of course08:17
@wikingthis would mean that if somebody doesn't have LAPACK08:17
@wikingmmm08:17
@wikingok no this is not the best idea08:17
@wikingsorry08:17
@wikingok them i'm just thinking about the naming of the08:17
@wikingfunction08:17
@wikingthe problem with these 4 letter namings of a function08:18
mikelingyep08:18
@wikingis that it's completely jibberish08:18
@wikingwhen you code08:18
mikelingsorry, what's the jibberish?08:18
@wikingcblas_dgemv08:19
@wikingor vvmp08:19
@wikingbtw08:19
@wikinglisitsyn, ping08:19
@wikingso i mean08:20
@wikingthis thing with08:20
mikelingah I see.08:20
mikelingthe vvmp just means *vector produce vector then plus matrix* which that function does08:21
@wikingyes08:22
@wikingi know but again08:22
@wikingthink about when you are coding08:22
@wikingthis is not intuitive08:22
@wikingjust as dgemv08:22
@wikingi mean lapack function has a rule as well08:22
@wikingfor naming08:22
@wikingthe first character is f = float or d = double08:22
mikelingmmm I have no idea about the rule.08:23
mikelingso you mean the dgemv is not some combination of random letters08:24
mikelingit do have pertinent meaning for that function08:25
mikelingmaybe we should use 'dgemv'? Just remove the cblas_ prefix?08:26
@wikingnono08:27
@wikingit's horrible08:27
@wikingi mean i hate these naming08:27
@wikingmikeling, http://www.netlib.org/lapack/lug/node24.html08:27
@wikingthere's the rule08:27
@wikingbut it's horrible08:27
@wikingjust a sec08:30
@wikingiin the meanwhile i have found some errors :)08:30
@wikingminor one08:30
@wikingjust commenting it into the pr08:30
mikelingalright, it's complex and I guess people won't get the point when they first look at the code08:31
mikelingok08:31
mikelingthank you :)08:31
@wikingok i've sent you the changes you need for the doxygen for sure08:33
@wikingok so you fix that part08:42
@wikingi'm just thinking now of the naming08:42
@wikingfor that those function08:42
-!- iglesiasg [~iglesiasg@217.119.234.214] has joined #shogun09:06
-!- mode/#shogun [+o iglesiasg] by ChanServ09:06
-!- geektoni [c1cdd248@gateway/web/freenode/ip.193.205.210.72] has joined #shogun09:28
@wikinggeektoni, you've pinged me yesterday?09:32
geektonihey wiking :) yeah, but I've already solved the problem.09:32
@wikingkk09:32
lisitsynwiking:10:01
lisitsyniglesiasg: hola senor10:01
@iglesiasglisitsyn: que pasa10:03
@wikinghi10:07
@wikinglisitsyn, btw10:24
@wikingiglesiasg,  as well10:24
@wikingany ideas how we should name these functions internally10:24
@wikingcblas_dgemv10:24
@wikingcblas_dger10:24
@wikingcblas_dgemm10:24
@wikingcblas_dgemv  is kind of a shift and scale of a Y vector10:24
@wikingsince one could take that A matrix is transforming the X vector into Y's space....10:25
@wikingbut yeah need better naming than dgemv10:25
@wiking:)10:25
lisitsynat least without cblas10:28
lisitsynbecause we call eigen there right?10:28
@wikinglinalg :)10:28
@wikingyeah of course we'll drop cblas_10:29
@wikingbut would be great if we could have a more intuitive naming10:29
@wikingthan zxvaseY10:29
@iglesiasgeigen-like naming?10:38
@iglesiasgShiftAndScale10:39
@iglesiasgAndDifferentiateAndNormalizeAndShiftAgain10:39
lisitsynFcukThenUnfcukItAgain10:40
lisitsynscnr10:40
@iglesiasghahaha10:41
@iglesiasgjajaja10:41
@wiking:10:46
@wiking:>10:46
@wikingbut yeah we need something10:46
@wikingnow10:46
@wiking:)10:46
lisitsynwiking: I am fine with dgemv10:47
lisitsynjust without cblas10:47
lisitsynI think these guys had some wisdom10:48
lisitsynI guess they had these_long_names and then decided to go with random sequences10:48
lisitsynat least it is googlable they said in 1970 :P10:49
@wiking:>10:53
@wikingit's not random :)10:53
@wikingmikeling, ok now you should have all the righ tinfo for that pr and you can finish it10:55
@wikingand then we can go back to the original PR )10:55
lisitsynwiking: youve been tricked10:56
@wiking?10:56
lisitsynit was random then they have come with reasonable explanation10:57
lisitsyndjfsjfsfj ok double integer field ….10:57
lisitsyn:P10:57
@wiking:)10:57
mikelingso we are going to use degmv? Just remove the cblas prefix11:01
@wikingyes11:07
@wikingindeed11:07
-!- geektoni [c1cdd248@gateway/web/freenode/ip.193.205.210.72] has quit [Quit: Page closed]11:58
@iglesiasgdo you guys know of any toolset for gdb that can do stuff like giving the max of an array?12:12
@iglesiasgI am debugging some code and have a long float array[len]. From gdb, I would like to do something like p max(array, len).12:13
lisitsynI don't12:18
@sukeyPull Request #3819 "make guassian get rid of LAPACK"  synchronized by MikeLing - https://github.com/shogun-toolbox/shogun/pull/381913:59
-!- geektoni [c1cdd24a@gateway/web/freenode/ip.193.205.210.74] has joined #shogun14:05
@wikinganybody around?14:46
@iglesiasg.o/15:04
@iglesiasgwiking: tell me15:04
@wikingalready fixed15:04
@wikingbut thnx15:04
@iglesiasg</315:04
mikelingwiking: mmm, it seems like the appveyor went wrong https://ci.appveyor.com/project/vigsterkr/shogun/build/4.2.763/job/ae4304tcat74e03716:04
mikelingbut no error comes out16:05
mikeling:/16:05
mikelingseems like the ci just run out of time and stopped. Maybe we could assume the code is good to go?16:10
-!- iglesiasg [~iglesiasg@217.119.234.214] has quit [Quit: leaving]16:59
mikelinglisitsyn: How could I retrigger ci if it accidentally error out? e.g https://ci.appveyor.com/project/vigsterkr/shogun/build/4.2.763/job/ae4304tcat74e03717:10
@wikingill do it now17:13
@sukeyPull Request #3819 "make guassian get rid of LAPACK"  merged by vigsterkr - https://github.com/shogun-toolbox/shogun/pull/381917:14
@sukeyNew Commit "Merge pull request #3819 from MikeLing/get_rid_of_LAPACK17:14
@sukeymake guassian get rid of LAPACK" to shogun-toolbox/shogun by vigsterkr: https://github.com/shogun-toolbox/shogun/commit/380c06f6d8ac74635bcca712e46c34bdd2d9c80c17:14
@wikingdone17:14
mikelingwiking: Thank you, I'm going to update related pr17:16
-!- geektoni [c1cdd24a@gateway/web/freenode/ip.193.205.210.74] has quit [Quit: Page closed]17:18
@sukeyPull Request #3821 "Add new constructor and get() method to Some<>."  merged by vigsterkr - https://github.com/shogun-toolbox/shogun/pull/382118:21
@sukeyNew Commit "Merge pull request #3821 from geektoni/patch-1818:21
@sukeyAdd new constructor and get() method to Some<>." to shogun-toolbox/shogun by vigsterkr: https://github.com/shogun-toolbox/shogun/commit/c2a9559dc2664c75649f913ab1bc72caa1a3e5a818:21
@sukeyPull Request #3812 "Add global fixture with binary label data"  synchronized by MikeLing - https://github.com/shogun-toolbox/shogun/pull/381218:24
@sukeyPull Request #3812 "Add global fixture with binary label data"  synchronized by MikeLing - https://github.com/shogun-toolbox/shogun/pull/381218:46
-!- mikeling [uid89706@gateway/web/irccloud.com/x-cohfsjksvtvnffvx] has quit [Quit: Connection closed for inactivity]21:10
-!- micmn_ [~micmn@ec2-54-67-24-92.us-west-1.compute.amazonaws.com] has joined #shogun21:36
-!- micmn [~micmn@ec2-54-67-24-92.us-west-1.compute.amazonaws.com] has quit [Ping timeout: 246 seconds]21:36
--- Log closed Thu May 25 00:00:44 2017
« Tuesday, 2017-05-23 Index Thursday, 2017-05-25 »

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