{"action":"created","issue":{"url":"https://api.github.com/repos/bartvm/blocks/issues/285","labels_url":"https://api.github.com/repos/bartvm/blocks/issues/285/labels{/name}","comments_url":"https://api.github.com/repos/bartvm/blocks/issues/285/comments","events_url":"https://api.github.com/repos/bartvm/blocks/issues/285/events","html_url":"https://github.com/bartvm/blocks/pull/285","id":57704407,"number":285,"title":"WIP: Switch from Dill to cPickle","user":{"login":"bartvm","id":2299595,"avatar_url":"https://avatars.githubusercontent.com/u/2299595?v=3","gravatar_id":"","url":"https://api.github.com/users/bartvm","html_url":"https://github.com/bartvm","followers_url":"https://api.github.com/users/bartvm/followers","following_url":"https://api.github.com/users/bartvm/following{/other_user}","gists_url":"https://api.github.com/users/bartvm/gists{/gist_id}","starred_url":"https://api.github.com/users/bartvm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bartvm/subscriptions","organizations_url":"https://api.github.com/users/bartvm/orgs","repos_url":"https://api.github.com/users/bartvm/repos","events_url":"https://api.github.com/users/bartvm/events{/privacy}","received_events_url":"https://api.github.com/users/bartvm/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-02-14T19:52:19Z","updated_at":"2015-02-14T21:15:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/bartvm/blocks/pulls/285","html_url":"https://github.com/bartvm/blocks/pull/285","diff_url":"https://github.com/bartvm/blocks/pull/285.diff","patch_url":"https://github.com/bartvm/blocks/pull/285.patch"},"body":"After spending a few hours going down the rabbit hole of the subtle differences in Python 2's and 3's data models, I think that I've made a tiny bit of progress... Just wondering what errors this will raise on Travis."},"comment":{"url":"https://api.github.com/repos/bartvm/blocks/issues/comments/74392255","html_url":"https://github.com/bartvm/blocks/pull/285#issuecomment-74392255","issue_url":"https://api.github.com/repos/bartvm/blocks/issues/285","id":74392255,"user":{"login":"bartvm","id":2299595,"avatar_url":"https://avatars.githubusercontent.com/u/2299595?v=3","gravatar_id":"","url":"https://api.github.com/users/bartvm","html_url":"https://github.com/bartvm","followers_url":"https://api.github.com/users/bartvm/followers","following_url":"https://api.github.com/users/bartvm/following{/other_user}","gists_url":"https://api.github.com/users/bartvm/gists{/gist_id}","starred_url":"https://api.github.com/users/bartvm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bartvm/subscriptions","organizations_url":"https://api.github.com/users/bartvm/orgs","repos_url":"https://api.github.com/users/bartvm/repos","events_url":"https://api.github.com/users/bartvm/events{/privacy}","received_events_url":"https://api.github.com/users/bartvm/received_events","type":"User","site_admin":false},"created_at":"2015-02-14T21:15:50Z","updated_at":"2015-02-14T21:15:50Z","body":"@rizar @dwf Tests are passing without Dill now! It might require a bit of cleaning up, but after trying to tackle all the serialization issues in 10 different ways, I ended up with going something so stupidly simple it had to work. Would like to know whether you guys think making this switch is a good idea.\r\n\r\n* You can't have references to functions outside of the global namespace lying around. This concerns functions before they're bound (i.e. when wrapping functions in decorators), but in Python 3 all class methods are also functions outside of the global namespace (because the \"unbound method\" doesn't exist anymore). Instead of doing something clever, I ended up storing the name of the function instead of the function itself. Note that this does mean that the function needs to stay part of the class where it was originally defined (so I move `apply` to `_apply` and then replace `apply` with an `Application` that looks up `_apply` when it needs it).\r\n* No more lambdas, anywhere. I personally don't mind that; makes for easier debugging."}}