{"action":"created","issue":{"url":"https://api.github.com/repos/GPflow/GPflow/issues/634","repository_url":"https://api.github.com/repos/GPflow/GPflow","labels_url":"https://api.github.com/repos/GPflow/GPflow/issues/634/labels{/name}","comments_url":"https://api.github.com/repos/GPflow/GPflow/issues/634/comments","events_url":"https://api.github.com/repos/GPflow/GPflow/issues/634/events","html_url":"https://github.com/GPflow/GPflow/pull/634","id":291609638,"number":634,"title":"Cross-covariance between different RBF kernels","user":{"login":"rhaps0dy","id":4928242,"avatar_url":"https://avatars3.githubusercontent.com/u/4928242?v=4","gravatar_id":"","url":"https://api.github.com/users/rhaps0dy","html_url":"https://github.com/rhaps0dy","followers_url":"https://api.github.com/users/rhaps0dy/followers","following_url":"https://api.github.com/users/rhaps0dy/following{/other_user}","gists_url":"https://api.github.com/users/rhaps0dy/gists{/gist_id}","starred_url":"https://api.github.com/users/rhaps0dy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhaps0dy/subscriptions","organizations_url":"https://api.github.com/users/rhaps0dy/orgs","repos_url":"https://api.github.com/users/rhaps0dy/repos","events_url":"https://api.github.com/users/rhaps0dy/events{/privacy}","received_events_url":"https://api.github.com/users/rhaps0dy/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2018-01-25T15:29:24Z","updated_at":"2018-05-02T11:34:54Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/GPflow/GPflow/pulls/634","html_url":"https://github.com/GPflow/GPflow/pull/634","diff_url":"https://github.com/GPflow/GPflow/pull/634.diff","patch_url":"https://github.com/GPflow/GPflow/pull/634.patch"},"body":"All the `tests/test_expectations.py` pass, of course including the one I added to test this.\r\n\r\nThere are a couple of things I don't quite like about my code, and hopefully you know of a better way to do them:\r\n  - Needing `nested_with`. That's because I needed to do `with params_as_tensors_for(a)` for every _different_ kernel and feature `a`, and you can't use `*map(params_as_tensors_for, unique_feat_kern)`.\r\n  - Having to write another implementation for `square_dist` starting on line 350. This is because the desired lengthscale to use `(La + Lb)` doesn't belong to any kernel, and also to use the default kernel's implementation one would need to pass `tf.sqrt(La + Lb)`, only to have it be squared again."},"comment":{"url":"https://api.github.com/repos/GPflow/GPflow/issues/comments/385947873","html_url":"https://github.com/GPflow/GPflow/pull/634#issuecomment-385947873","issue_url":"https://api.github.com/repos/GPflow/GPflow/issues/634","id":385947873,"user":{"login":"ialong","id":5802525,"avatar_url":"https://avatars0.githubusercontent.com/u/5802525?v=4","gravatar_id":"","url":"https://api.github.com/users/ialong","html_url":"https://github.com/ialong","followers_url":"https://api.github.com/users/ialong/followers","following_url":"https://api.github.com/users/ialong/following{/other_user}","gists_url":"https://api.github.com/users/ialong/gists{/gist_id}","starred_url":"https://api.github.com/users/ialong/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ialong/subscriptions","organizations_url":"https://api.github.com/users/ialong/orgs","repos_url":"https://api.github.com/users/ialong/repos","events_url":"https://api.github.com/users/ialong/events{/privacy}","received_events_url":"https://api.github.com/users/ialong/received_events","type":"User","site_admin":false},"created_at":"2018-05-02T11:34:53Z","updated_at":"2018-05-02T11:34:53Z","author_association":"CONTRIBUTOR","body":"It’s not yet supported and it would still be useful, but the code should be updated to fit in with the new sped-up expectations\n\n> On 2 May 2018, at 12:29, Mark van der Wilk <notifications@github.com> wrote:\n> \n> @vdutor / @ialong is this already supported, or is this still useful? Feel free to close the issue if it's no longer relevant.\n> \n> —\n> You are receiving this because you were mentioned.\n> Reply to this email directly, view it on GitHub, or mute the thread.\n> \n"}}
{"actor":{"display_login":"ialong"}}