IRC logs of #shogun for Friday, 2018-04-20

--- Log opened Fri Apr 20 00:00:32 2018
-shogun-buildbot:#shogun- Build nightly_bsd_license #140 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/4/builds/14003:31
-shogun-buildbot:#shogun- Build nightly_default #128 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/17/builds/12806:31
-!- sukey [~nodebot@ks312251.kimsufi.com] has joined #shogun10:10
-!- mode/#shogun [+o sukey] by ChanServ10:10
-!- witness [uid10044@gateway/web/irccloud.com/x-euzmyvhpfazcmxpc] has joined #shogun13:17
@wikinglisitsyn, would you mind if we make AnyParameter get_paramter() public?15:19
@wikingor you rather have a visit() for SGO?15:20
@wikingnevermind15:20
@wikingi'll move it to sgo15:21
@wikingthe whole serialization thing :)15:21
@wikingnoooo i dont wanna do interfacing !@#r415:28
@wikingi must say that this is the most cumbersome design that we could have, because either one does serialization with observing the object -> need to make the deserializer and serializer a friend class of SGO or we need to have an elaborate interface for all the different serialization format :P15:30
@wikingi dont like the observing thing really because that assumes that everything is public of the object that is required for serialization15:31
@wikingbut i dont really like writing a bunch of wrapper code just for the sake of doing it :)15:31
lisitsynwiking: hey16:45
@wikingho16:45
lisitsynwiking: so you have some troubles exposing the data?16:46
lisitsynI actually don't mind exposing whatever sgobject has16:46
@wikingmmm16:46
@wikingbut you get it that here16:46
@wikingwe would be fucking16:46
@wikingwith the encapsulation pattern16:46
@wikingright?16:46
@wikingbecause you kind of force to expose *everythong16:46
@wikingof sgo16:46
@wikinganyparameter16:46
@wikingetc etc16:46
@wikinglisitsyn, i'll show u in a copy paste how super ugly this shit becomes17:05
lisitsynwiking: ok17:06
@wikingno man this is even more shittier than you think :)17:07
@wikinglisitsyn, https://pastebin.com/aDspYAaE17:10
@wikingactually17:12
@wikingthis wont even work :)(17:12
@wikingand note that this is not SAX but DOM17:13
@wikingSAX reading woould be a suicide mission17:13
@wikingbut this doesn't let you do any thing like17:13
@wikingserialize more than 1 object17:13
@wikinginto the file17:13
@wiking:D17:13
@wikingok so forget it17:14
@wikingthis is even more and more worse17:14
@wikingok this wont work with dom17:14
@wikingso we need sax17:14
@wikingand if you need SAX17:14
@wikingit's a mayham17:14
lisitsynwiking: yeah input is messy so far17:21
lisitsynoutput was easier indeed17:21
@wikinglisitsyn, https://pastebin.com/4DUVTHmH17:22
@wikingthis should work imo17:22
lisitsynyes looks good enough17:23
lisitsynbut not input17:23
@wikingas well as the problem of encapuslation :P17:26
@wikingnote the story of SAX in reading17:28
@wikingthat's even more horrible17:28
@wikingas you have to implement a full state machine :)))17:28
@wikingthis is just wasting LoC :)17:28
@wikingthat's my opinion17:28
@wikingnote that here we start to have the same story... as soon as we start to refactor the anything in the library17:29
@wikingeverything will break with serialization17:29
@wikingbecause of the fact that we dont implement the serialization within the class itself17:29
@wikingand this will need to be refactored everywhere17:30
@wikingmeaning in every serialization format17:30
@wikingof course the other way round is getting shittier17:31
@wikingas that means that you need to interface a serialization interface with primitives17:31
@wiking.write(bool) .write(int) .write17:31
@wikingwhich is getting ridicolous especially that the same shit has to be implemented in the visitor17:31
@wikingtada17:31
@wiking:)17:31
@wikingbtw17:36
@wikingi'm pretty sure that with the current design17:36
@wikingyou just cannot do ser/deser :)17:36
@wikingas this is the 10th iteration of this shit17:36
@wikingand we are still the same point17:37
@wikingi mean if we do it as an external case, i.e. your serialization fw is 'visiting' every method (so you need to handcraft everything for obj) then you have the current shitoooo situation that you see in the pasted code17:39
@wikingnamely when you start to read in recursively it kabooms17:39
lisitsynwiking: yeah but I don't really get what's exactly the problem in the design17:39
@wikingnow the other story ... namely that you let the objs drive serialization17:39
@wikingyou get where feature/cereal is being stuck17:39
@wiking:)17:39
@wikingso mumimu17:40
@wikingthis is asdf17:40
lisitsynwiking: who should drive the thing then?17:41
@wikingthere's a good reason17:41
@wikingwhy everybody lets this specified by the class/obj17:41
@wikingnamely that you know yourself17:41
@wikingwhat you need for getting back your state17:41
@wikingjust as in case of copy-ctor17:42
@wikingbtw the saddest thing of this all is that we are doing here something17:42
@wikingthat has been done by billions of times17:42
@wiking:)))17:42
@wikinghttp://rapidjson.org/md_doc_sax.html#Reader17:43
@wikinglook here what you need for the SAX parser17:43
@wiking:)17:43
@wikingand now there you need to implement a f-ing crazy FSM that you read in first the f-ing tuple for the f-ing sgobject to call the f-ing factory and once you have that you have to do the f-ing filling of the f-ing obj but then you arrive to the f-ing StartArray and you dont even know there the f-ing type of the f-ing array because then you still need to wait at least a next call from the input so that you kow the type of SGVector oooor SGMatrix you wanna17:46
@wiking read in17:46
@wikingman this is like loco17:46
@wikingit's like 2002 again17:46
@wikingwhere i parsed xml with libxml2 sax parser17:46
@wikingand you know exactly17:47
@wikingthat this shit will somewhere just again blow up17:47
lisitsyn?\_(?)_/?17:47
@wikingyeah exactly17:47
@wikingi dont get it17:47
@wikingwhy do we have to take this design17:47
@wikingthat makes the whole fucking shit17:47
@wikingjust like another piece of shit17:48
@wiking:)17:48
lisitsynI don't really get what's the design in this context17:48
lisitsynkey value? methods?17:48
@wikingwhich case?17:48
lisitsynI mean you say design is broken17:48
lisitsynto unbroke we need what exactly is17:48
@wikingyeah something in the design forces me to do things17:48
@wikingthat i cannot do17:48
@wikingimo how we create objs17:49
@wikingit's somewhere there17:49
lisitsynclassifier('blabla') is bad?17:49
@wikingnono17:49
lisitsynbut plugins?17:49
lisitsynwhat is?17:49
@wikingcreate("asdf", geenric_type)17:49
@wikingbecause that assumes that you can inspect17:49
@wikingor peek17:50
@wikingthe sax17:50
@wikingbecause you need sax17:50
@wikingas otherwise it's even more kaboom17:50
@wiking:)17:50
lisitsynyes, sax17:50
@wikinglisitsyn, https://www.nytimes.com/2018/04/19/opinion/standing-up-at-your-desk-could-make-you-smarter.html17:57
lisitsynwiking: doctor did not give me any prescriptions17:58
@wikingshow him this :)17:58
lisitsynshe said like there is no such prescription per se17:58
@wikingfucker17:58
@wikingshe just wanted more money :)17:58
lisitsynwiking: I asked my supply team what should I do to get a standing desk17:59
lisitsyn:D17:59
@wiking:)17:59
lisitsynwiking: apart from breaking my spine18:00
@wikingnoup18:02
@wikingi thikn i already have a proof18:02
@wikingthat this cannnot be deserialized18:02
@wiking:D18:02
@wikingso i'18:11
@wikingm out :)18:11
@wikingworking on something more fruitful :)18:11
@wikingsorry18:11
@wikingttyl18:11
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/426218:19
@sukey[https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4262 merged by iglesias18:19
@sukey[https://github.com/shogun-toolbox/shogun] iglesias pushed 2 commits:18:19
@sukeyhttps://github.com/shogun-toolbox/shogun/commit/b63e281ba37ca227d54036a924a445a47ed737e618:19
@sukeyhttps://github.com/shogun-toolbox/shogun/commit/2102ffedefbcff7afe9ce98e28e4a90dd6ba856918:19
-!- iglesias [50fe9643@gateway/web/freenode/ip.80.254.150.67] has joined #shogun18:20
iglesiaswiking: just checking out now that ny times article18:26
iglesiasthey went all out with this one "The implication is that the more time you spend in a chair the worse it is for your brain health, resulting in possible impairment in learning and memory." wow18:27
-!- iglesias [50fe9643@gateway/web/freenode/ip.80.254.150.67] has quit [Quit: Page closed]18:35
-shogun-buildbot:#shogun- Build deb1 - libshogun #393 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/10/builds/39318:51
-shogun-buildbot:#shogun- Build yakkety - libshogun #450 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/1/builds/45019:30
-!- travis-ci [~travis-ci@ec2-54-162-158-152.compute-1.amazonaws.com] has joined #shogun19:43
travis-ciit's Fernando J. Iglesias Garcia'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/36920895919:43
-!- travis-ci [~travis-ci@ec2-54-162-158-152.compute-1.amazonaws.com] has left #shogun []19:43
-shogun-buildbot:#shogun- Build deb3 - interfaces #315 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/37/builds/31519:51
-!- witness [uid10044@gateway/web/irccloud.com/x-euzmyvhpfazcmxpc] has quit [Quit: Connection closed for inactivity]21:37
--- Log closed Sat Apr 21 00:00:33 2018

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