--- Log opened Tue Apr 17 00:00:28 2018 | ||
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4168 synchronized by syashakash | 03:07 |
---|---|---|
-!- travis-ci [~travis-ci@ec2-54-221-31-242.compute-1.amazonaws.com] has joined #shogun | 04:35 | |
travis-ci | it's syashakash's turn to pay the next round of drinks for the massacre he caused in syashakash/shogun: https://travis-ci.org/syashakash/shogun/builds/367443024 | 04:35 |
-!- travis-ci [~travis-ci@ec2-54-221-31-242.compute-1.amazonaws.com] has left #shogun [] | 04:35 | |
-shogun-buildbot:#shogun- Build nightly_bsd_license #137 is complete: Success [build successful] - http://buildbot.shogun-toolbox.org:8080/#builders/4/builds/137 | 06:03 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4236 synchronized by shubham808 | 06:23 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4236 synchronized by shubham808 | 06:25 |
-shogun-buildbot:#shogun- Build nightly_default #125 is complete: Failure [failed test (failure)] - http://buildbot.shogun-toolbox.org:8080/#builders/17/builds/125 | 07:09 | |
-!- travis-ci [~travis-ci@ec2-107-21-150-51.compute-1.amazonaws.com] has joined #shogun | 07:13 | |
travis-ci | it's Shubham Shukla's turn to pay the next round of drinks for the massacre he caused in shubham808/shogun: https://travis-ci.org/shubham808/shogun/builds/367480533 | 07:14 |
-!- travis-ci [~travis-ci@ec2-107-21-150-51.compute-1.amazonaws.com] has left #shogun [] | 07:14 | |
-!- travis-ci [~travis-ci@ec2-107-21-150-51.compute-1.amazonaws.com] has joined #shogun | 07:26 | |
travis-ci | it's Shubham Shukla's turn to pay the next round of drinks for the massacre he caused in shubham808/shogun: https://travis-ci.org/shubham808/shogun/builds/367480533 | 07:26 |
-!- travis-ci [~travis-ci@ec2-107-21-150-51.compute-1.amazonaws.com] has left #shogun [] | 07:26 | |
-!- iglesias [~iglesias@77.243.24.142] has joined #shogun | 09:01 | |
-!- HeikoS [~heiko@77.243.24.142] has joined #shogun | 09:11 | |
-!- mode/#shogun [+o HeikoS] by ChanServ | 09:11 | |
@wiking | lisitsyn, yo | 09:43 |
@wiking | lisitsyn, one question that i still dont get... how do you get the type for a tag? | 09:44 |
@wiking | say i have LibSVM and have its tags | 09:46 |
@wiking | and the promise for me was that if i traverse the tags i know exactly what type i'm gonna be reading in :) | 09:48 |
lisitsyn | wiking: no but Any can inject its value into visitor | 09:53 |
lisitsyn | that's the whole point of visitors | 09:53 |
lisitsyn | but you can't know outside | 09:53 |
@wiking | yeye | 09:53 |
@wiking | i'm talking now reversing | 09:53 |
@wiking | i.e reading into any | 09:54 |
@wiking | the visitor i have and that works nicely | 09:54 |
@wiking | the problemm is the other way around | 09:54 |
@wiking | same as in case of cereal | 09:54 |
@wiking | :) | 09:54 |
lisitsyn | aah | 09:54 |
lisitsyn | wiking: uh :) | 09:55 |
@wiking | i mean the problem is that | 09:55 |
lisitsyn | wiking: you may obtain type_info | 09:55 |
lisitsyn | from any | 09:55 |
@wiking | this is where i've got stuck | 09:55 |
lisitsyn | and then switch over it | 09:55 |
@wiking | yeye | 09:55 |
@wiking | but then we are where we were with cereal :)))) | 09:55 |
lisitsyn | no, why? | 09:56 |
@wiking | afaik type_info is not consistent | 09:56 |
@wiking | over machines | 09:56 |
@wiking | or libstdc++ | 09:56 |
lisitsyn | not sure | 09:56 |
-!- HeikoS [~heiko@77.243.24.142] has quit [Ping timeout: 260 seconds] | 09:56 | |
lisitsyn | wiking: ok better solution then | 09:56 |
lisitsyn | wiking: visitor but that provides reference | 09:56 |
@wiking | There is no guarantee that the same std::type_info instance will be referred to by all evaluations of the typeid expression on the same type, although std::type_info::hash_code of those type_info objects would be identical, as would be their std::type_index. | 09:56 |
@wiking | There is no guarantee that the same std::type_info instance will be referred to by all evaluations of the typeid expression on the same type, although std::type_info::hash_code of those type_info objects would be identical, as would be their std::type_index. | 09:56 |
@wiking | sorry for the double copy | 09:56 |
lisitsyn | like visit(T& value) { value = read(..) } | 09:57 |
@wiking | aha | 09:57 |
lisitsyn | see what I mean? | 09:57 |
@wiking | compiling | 09:57 |
@wiking | just a sec | 09:57 |
@wiking | :) | 09:57 |
@wiking | ok so then we have 2 type of visitor? | 09:57 |
@wiking | one with const T | 09:57 |
@wiking | and one with T& ? | 09:58 |
lisitsyn | or just merge them into one | 09:58 |
lisitsyn | which is also fine to me | 09:58 |
lisitsyn | I mean T& always is as good | 09:58 |
@wiking | i mean actually | 09:58 |
@wiking | virtual void on(const bool*) = 0; | 09:58 |
@wiking | virtual void on(const int32_t*) = 0; | 09:58 |
@wiking | so if we'd drop const | 09:58 |
@wiking | it would work no? | 09:58 |
@wiking | i mean because it's a pointer | 09:58 |
lisitsyn | wiking: exactly | 09:59 |
lisitsyn | yeah we already using pointers | 09:59 |
lisitsyn | so just drop the const and that's it | 09:59 |
@wiking | kkk but lemme think | 09:59 |
lisitsyn | wiking: in the implementation you will be calling appropriate methods to read bools, read vectors, etc | 10:00 |
@wiking | yeye | 10:00 |
@wiking | mmm ok so i have VisitorReader | 10:00 |
@wiking | and VisitorWriter | 10:00 |
lisitsyn | or ReadVisitor WriteVisitor | 10:01 |
-!- iglesias [~iglesias@77.243.24.142] has quit [Ping timeout: 256 seconds] | 10:14 | |
-!- ricklly_ [~ricklly@2001:cc0:2020:4017:cde3:b86a:f647:f008] has joined #shogun | 10:24 | |
-!- HeikoS [~heiko@77.243.24.142] has joined #shogun | 11:22 | |
-!- mode/#shogun [+o HeikoS] by ChanServ | 11:22 | |
-!- iglesias [~iglesias@77.243.24.142] has joined #shogun | 11:25 | |
-!- ricklly_ [~ricklly@2001:cc0:2020:4017:cde3:b86a:f647:f008] has quit [Remote host closed the connection] | 11:56 | |
@sukey | [https://github.com/shogun-toolbox/shogun] vigsterkr pushed 2 commits: | 12:04 |
@sukey | https://github.com/shogun-toolbox/shogun/commit/7bbf0977daa1e8af6f4a01a40234f3351739a41e | 12:04 |
@sukey | https://github.com/shogun-toolbox/shogun/commit/67bf8100ce7567921c0ac1c2f91a5a5fb218baf1 | 12:04 |
-!- travis-ci [~travis-ci@ec2-54-144-195-124.compute-1.amazonaws.com] has joined #shogun | 12:12 | |
travis-ci | it's Viktor Gal'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/367576421 | 12:12 |
-!- travis-ci [~travis-ci@ec2-54-144-195-124.compute-1.amazonaws.com] has left #shogun [] | 12:12 | |
lisitsyn | wiking: O_o :) | 12:28 |
lisitsyn | so you go with cereal and visitors? | 12:28 |
@wiking | it's just a quick check | 12:31 |
@wiking | if that'd work | 12:31 |
@wiking | as i had the same problem there | 12:32 |
@wiking | that we have here | 12:32 |
@wiking | lisitsyn, and there i had the saving working nicely already | 12:32 |
@wiking | so now i just test it on hedonism | 12:32 |
lisitsyn | wiking: I think this is going to work | 12:32 |
@wiking | would that work the other way round | 12:32 |
lisitsyn | with that we don't need to write json/xml/etc things | 12:33 |
@wiking | yeah lets see | 12:33 |
@wiking | just trying to get it to worj | 12:33 |
@wiking | work | 12:33 |
lisitsyn | wiking: one thing I am not sure | 12:42 |
lisitsyn | is that Archive is templated thing | 12:42 |
lisitsyn | it is not going to work directly with swig | 12:42 |
lisitsyn | so we still wrap it | 12:42 |
@wiking | ah yeah | 12:42 |
@wiking | but we do a trick there | 12:42 |
lisitsyn | wiking: it might make sense to implement CerealSerializer, CerealDeserializer | 12:42 |
@wiking | we extend sgobj | 12:42 |
@wiking | to have those | 12:43 |
lisitsyn | so that Cereaal is not the dependency but just one way of doing that | 12:43 |
@wiking | ic | 12:43 |
@wiking | yeah we can mix that | 12:43 |
@wiking | just i'm still testing this whole idea | 12:43 |
@wiking | (compiling) | 12:43 |
@sukey | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/d0d526d76d96a7939b6cd2e558a80a2ddebc9ae9 by vigsterkr | 12:58 |
@wiking | still far from the good :( | 12:58 |
@wiking | lisitsyn, we have aw problem | 13:23 |
@wiking | :)))) | 13:23 |
lisitsyn | houston | 13:23 |
lisitsyn | yeah? | 13:23 |
-!- travis-ci [~travis-ci@ec2-54-221-31-242.compute-1.amazonaws.com] has joined #shogun | 13:23 | |
travis-ci | it's Viktor Gal'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/367597200 | 13:23 |
-!- travis-ci [~travis-ci@ec2-54-221-31-242.compute-1.amazonaws.com] has left #shogun [] | 13:23 | |
@wiking | lisitsyn, i think when i do ReadVisitor | 13:24 |
@wiking | i go to the on(Empty<>() | 13:24 |
@wiking | of the visitor | 13:24 |
@wiking | because the object is empty... it only has been created with default ctor | 13:24 |
@wiking | but i'm not 1005 | 13:24 |
@wiking | 100% | 13:24 |
@wiking | just debugging | 13:24 |
-!- travis-ci [~travis-ci@ec2-54-144-195-124.compute-1.amazonaws.com] has joined #shogun | 13:27 | |
travis-ci | it's Viktor Gal'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/367597200 | 13:27 |
-!- travis-ci [~travis-ci@ec2-54-144-195-124.compute-1.amazonaws.com] has left #shogun [] | 13:27 | |
@wiking | coz any visitor has void on(Empty*) | 13:27 |
@wiking | { | 13:27 |
@wiking | } | 13:27 |
@wiking | void on(...) | 13:27 |
@wiking | { | 13:27 |
@wiking | } | 13:27 |
@wiking | aaah nooo fuck | 13:29 |
@wiking | i have a hunch | 13:29 |
@wiking | that somewhere i left in the const | 13:29 |
@wiking | argh | 13:31 |
@wiking | template <class T> | 13:31 |
@wiking | inline T const* typed_pointer(const void* ptr) | 13:31 |
@wiking | { | 13:31 |
@wiking | return static_cast<T const*>(ptr); | 13:31 |
@wiking | } | 13:31 |
@wiking | lisitsyn, basically because our visitor is not on(T* v) | 13:32 |
@wiking | we need const removal :) | 13:33 |
lisitsyn | ah | 13:33 |
lisitsyn | ok | 13:33 |
lisitsyn | well fixable | 13:33 |
@wiking | lisitsyn, visitor->on(const_cast<T>(typed_pointer<T>(storage))); | 13:33 |
@wiking | or rather have a different typed_pointer wrapper? | 13:33 |
@wiking | just testing with the above fix | 13:34 |
@wiking | if that's the case | 13:34 |
lisitsyn | const cast is fine | 13:34 |
@wiking | testing | 13:36 |
@wiking | man i compiled like 10 times | 13:36 |
@wiking | the whole lib | 13:36 |
@wiking | lisitsyn, ok it matched all the serialization part | 13:43 |
@wiking | but when i do visit on an empty class | 13:44 |
@wiking | as if no visitor is being called | 13:45 |
lisitsyn | wiking: empty sgobject? | 13:46 |
lisitsyn | or? | 13:47 |
@wiking | i mean it's a default ctored object | 13:47 |
@wiking | but i'm still debugging | 13:49 |
@wiking | i might know wtf | 13:53 |
-!- iglesiasg [~iglesias@77.243.26.98] has joined #shogun | 14:13 | |
-!- HeikoS [~heiko@77.243.24.142] has quit [Read error: Connection reset by peer] | 14:13 | |
-!- iglesias [~iglesias@77.243.24.142] has quit [Read error: Connection reset by peer] | 14:13 | |
@wiking | mmm ok i have no clue why that doesn't work lisitsyn ... but basically SGObject's serialization method is being called good and it actually does a serizalization properly | 14:19 |
@wiking | the other way round doesn't work | 14:19 |
lisitsyn | wiking: what exactly fails? | 14:21 |
@wiking | SGObject's cereal functions not being called | 14:21 |
lisitsyn | wiking: can you point me to the code? | 14:21 |
@wiking | save works good | 14:21 |
@wiking | yeah sure | 14:21 |
@wiking | this is called https://github.com/shogun-toolbox/shogun/blob/feature/cereal/src/shogun/base/SGObject.h#L149 | 14:22 |
@wiking | this is not https://github.com/shogun-toolbox/shogun/blob/feature/cereal/src/shogun/base/SGObject.h#L167 | 14:22 |
@wiking | and the unit test | 14:22 |
@wiking | https://github.com/shogun-toolbox/shogun/blob/feature/cereal/tests/unit/io/Cereal_unittest.cc#L176 | 14:22 |
lisitsyn | hmm | 14:24 |
@wiking | and the json has this | 14:24 |
@wiking | https://pastebin.com/kDvAdTbu | 14:25 |
-!- ricklly_ [~ricklly@2001:cc0:2020:4017:cde3:b86a:f647:f008] has joined #shogun | 14:25 | |
-!- HeikoS [~heiko@77.243.26.98] has joined #shogun | 14:29 | |
-!- mode/#shogun [+o HeikoS] by ChanServ | 14:29 | |
-!- ricklly_ [~ricklly@2001:cc0:2020:4017:cde3:b86a:f647:f008] has quit [Remote host closed the connection] | 14:54 | |
-!- iglesiasg [~iglesias@77.243.26.98] has quit [Ping timeout: 264 seconds] | 15:39 | |
@wiking | lisitsyn, there's no way i can match a template with the return value one expects right? so say template<typename T> T asdf() {...} and then i have a = asdf() where a has a type | 17:08 |
@wiking | of X or whatever | 17:08 |
-!- iglesiasg [~iglesias@77.243.26.98] has joined #shogun | 17:22 | |
@sukey | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/4f2d50669800e5b3f375497413f76508343bc8d3 by vigsterkr | 17:57 |
@wiking | lisitsyn, ^ this almost works...have some problem with class_list.h inclusion | 17:57 |
@wiking | other than that it should work | 17:57 |
-!- travis-ci [~travis-ci@ec2-54-147-20-153.compute-1.amazonaws.com] has joined #shogun | 18:06 | |
travis-ci | it's Viktor Gal'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/367723870 | 18:06 |
-!- travis-ci [~travis-ci@ec2-54-147-20-153.compute-1.amazonaws.com] has left #shogun [] | 18:06 | |
lisitsyn | wiking: cool | 18:15 |
@wiking | ok works | 18:17 |
@wiking | ok now i do the last test | 18:17 |
@wiking | and then i push | 18:17 |
@wiking | lisitsyn, here? | 19:39 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4261 opened by karlnapf | 19:57 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4261 synchronized by karlnapf | 20:04 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4262 opened by iglesias | 20:13 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4262 | 20:13 |
-!- iglesiasg [~iglesias@77.243.26.98] has quit [Remote host closed the connection] | 20:17 | |
-!- iglesiasg [~iglesias@77.243.26.98] has joined #shogun | 20:17 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Issue https://github.com/shogun-toolbox/shogun/issues/3975 closed by iglesias | 20:27 |
-!- travis-ci [~travis-ci@ec2-54-147-20-153.compute-1.amazonaws.com] has joined #shogun | 20:27 | |
travis-ci | it's syashakash's turn to pay the next round of drinks for the massacre he caused in syashakash/shogun: https://travis-ci.org/syashakash/shogun/builds/367756704 | 20:27 |
-!- travis-ci [~travis-ci@ec2-54-147-20-153.compute-1.amazonaws.com] has left #shogun [] | 20:27 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4261 synchronized by karlnapf | 20:27 |
@sukey | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/ca0f19cd0c3db7d3ca36ed9ee6201ef317ae89f8 by vigsterkr | 20:33 |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4235 synchronized by shubham808 | 20:37 |
-!- travis-ci [~travis-ci@ec2-54-92-178-164.compute-1.amazonaws.com] has joined #shogun | 20:42 | |
travis-ci | it's syashakash's turn to pay the next round of drinks for the massacre he caused in syashakash/shogun: https://travis-ci.org/syashakash/shogun/builds/367756704 | 20:42 |
-!- travis-ci [~travis-ci@ec2-54-92-178-164.compute-1.amazonaws.com] has left #shogun [] | 20:42 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4261 synchronized by karlnapf | 20:45 |
@sukey | [https://github.com/shogun-toolbox/shogun] New branch feature/prune_swig created | 20:47 |
@sukey | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/6475cf9956bc7e920e40b71836398161ec71ff5f by karlnapf | 20:47 |
-!- iglesiasg [~iglesias@77.243.26.98] has quit [Ping timeout: 240 seconds] | 21:46 | |
-!- travis-ci [~travis-ci@ec2-54-147-20-153.compute-1.amazonaws.com] has joined #shogun | 21:55 | |
travis-ci | it's Shubham Shukla's turn to pay the next round of drinks for the massacre he caused in shubham808/shogun: https://travis-ci.org/shubham808/shogun/builds/367796144 | 21:55 |
-!- travis-ci [~travis-ci@ec2-54-147-20-153.compute-1.amazonaws.com] has left #shogun [] | 21:55 | |
-!- travis-ci [~travis-ci@ec2-54-92-178-164.compute-1.amazonaws.com] has joined #shogun | 22:18 | |
travis-ci | it's Shubham Shukla's turn to pay the next round of drinks for the massacre he caused in shubham808/shogun: https://travis-ci.org/shubham808/shogun/builds/367796144 | 22:18 |
-!- travis-ci [~travis-ci@ec2-54-92-178-164.compute-1.amazonaws.com] has left #shogun [] | 22:18 | |
@sukey | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/2373affaa3806b9023e87d5479a7e3c5366d944c by vigsterkr | 22:24 |
@sukey | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/2c04e597035d7d24b35240020a6240a827170495 by vigsterkr | 22:26 |
-!- travis-ci [~travis-ci@ec2-54-92-178-164.compute-1.amazonaws.com] has joined #shogun | 22:41 | |
travis-ci | it's Viktor Gal'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/367794032 | 22:41 |
-!- travis-ci [~travis-ci@ec2-54-92-178-164.compute-1.amazonaws.com] has left #shogun [] | 22:41 | |
-!- iglesiasg [~iglesias@77.243.26.98] has joined #shogun | 22:42 | |
-!- iglesiasg [~iglesias@77.243.26.98] has quit [Ping timeout: 268 seconds] | 22:50 | |
-!- iglesiasg [~iglesias@77.243.26.98] has joined #shogun | 22:52 | |
-!- iglesiasg [~iglesias@77.243.26.98] has quit [Ping timeout: 256 seconds] | 23:01 | |
-!- iglesiasg [~iglesias@77.243.26.98] has joined #shogun | 23:12 | |
@sukey | [https://github.com/shogun-toolbox/shogun] Pull Request https://github.com/shogun-toolbox/shogun/pull/4262 synchronized by iglesias | 23:14 |
-!- iglesiasg [~iglesias@77.243.26.98] has quit [Client Quit] | 23:15 | |
-!- iglesiasg [~iglesias@77.243.26.98] has joined #shogun | 23:15 | |
@sukey | [https://github.com/shogun-toolbox/shogun] New commit https://github.com/shogun-toolbox/shogun/commit/41dabf91db36ddccfeb4ac9f6004ba5c168c8618 by vigsterkr | 23:20 |
-!- iglesiasg [~iglesias@77.243.26.98] has quit [Remote host closed the connection] | 23:26 | |
-!- iglesiasg [~iglesias@77.243.26.98] has joined #shogun | 23:26 | |
@sukey | [https://github.com/shogun-toolbox/shogun] New branch w/streams-serialization created | 23:32 |
@sukey | [https://github.com/shogun-toolbox/shogun] vigsterkr pushed 8 commits: | 23:32 |
@sukey | https://github.com/shogun-toolbox/shogun/commit/fac2fef3ec600b8b4ec7f899dd3bb66257c09cdb | 23:32 |
@sukey | https://github.com/shogun-toolbox/shogun/commit/1c6256fd3c76ea6a37b414f65c8e6aeefbad3b56 | 23:32 |
@sukey | https://github.com/shogun-toolbox/shogun/commit/d71bda8b01014c96030fc060d6640720c91605fb | 23:32 |
@sukey | https://github.com/shogun-toolbox/shogun/commit/a6ed4bd9b7d2285625a02d2b94adc9cf40608b41 | 23:32 |
@sukey | https://github.com/shogun-toolbox/shogun/commit/e1c3ffe691c18dd5aa9c36e2fceba5a2dbe01725 | 23:32 |
@sukey | https://github.com/shogun-toolbox/shogun/commit/9dabacaeab7e24455cd42aec59dda8489e518738 | 23:32 |
-!- sukey [~nodebot@ks312251.kimsufi.com] has quit [Remote host closed the connection] | 23:32 | |
--- Log closed Wed Apr 18 00:00:29 2018 |
Generated by irclog2html.py 2.10.0 by Marius Gedminas - find it at mg.pov.lt!