{"action":"created","issue":{"url":"https://api.github.com/repos/SignalK/specification/issues/314","repository_url":"https://api.github.com/repos/SignalK/specification","labels_url":"https://api.github.com/repos/SignalK/specification/issues/314/labels{/name}","comments_url":"https://api.github.com/repos/SignalK/specification/issues/314/comments","events_url":"https://api.github.com/repos/SignalK/specification/issues/314/events","html_url":"https://github.com/SignalK/specification/pull/314","id":198238844,"number":314,"title":"Clarify context and allow for update-specific contexts","user":{"login":"jboynes","id":1704738,"avatar_url":"https://avatars.githubusercontent.com/u/1704738?v=3","gravatar_id":"","url":"https://api.github.com/users/jboynes","html_url":"https://github.com/jboynes","followers_url":"https://api.github.com/users/jboynes/followers","following_url":"https://api.github.com/users/jboynes/following{/other_user}","gists_url":"https://api.github.com/users/jboynes/gists{/gist_id}","starred_url":"https://api.github.com/users/jboynes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jboynes/subscriptions","organizations_url":"https://api.github.com/users/jboynes/orgs","repos_url":"https://api.github.com/users/jboynes/repos","events_url":"https://api.github.com/users/jboynes/events{/privacy}","received_events_url":"https://api.github.com/users/jboynes/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-12-31T16:40:55Z","updated_at":"2017-01-01T22:25:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/SignalK/specification/pulls/314","html_url":"https://github.com/SignalK/specification/pull/314","diff_url":"https://github.com/SignalK/specification/pull/314.diff","patch_url":"https://github.com/SignalK/specification/pull/314.patch"},"body":"Allow `context` to be supplied as a object rather than a `.` delimited string (while still supporting the older format for backwards compatibility).\r\n\r\nAllow `context` to be specified for each individual update to allow deltas to span multiple contexts (e.g. updating all vessels in a race or commercial fleet).\r\n\r\nProposes an alternative representation for an update as a simple object rather than path/value pairs. The two forms can be distinguished by the type of the `values` property making this a backward compatible, additive change."},"comment":{"url":"https://api.github.com/repos/SignalK/specification/issues/comments/269922417","html_url":"https://github.com/SignalK/specification/pull/314#issuecomment-269922417","issue_url":"https://api.github.com/repos/SignalK/specification/issues/314","id":269922417,"user":{"login":"rob42","id":3239261,"avatar_url":"https://avatars.githubusercontent.com/u/3239261?v=3","gravatar_id":"","url":"https://api.github.com/users/rob42","html_url":"https://github.com/rob42","followers_url":"https://api.github.com/users/rob42/followers","following_url":"https://api.github.com/users/rob42/following{/other_user}","gists_url":"https://api.github.com/users/rob42/gists{/gist_id}","starred_url":"https://api.github.com/users/rob42/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rob42/subscriptions","organizations_url":"https://api.github.com/users/rob42/orgs","repos_url":"https://api.github.com/users/rob42/repos","events_url":"https://api.github.com/users/rob42/events{/privacy}","received_events_url":"https://api.github.com/users/rob42/received_events","type":"User","site_admin":false},"created_at":"2017-01-01T22:25:45Z","updated_at":"2017-01-01T22:25:45Z","body":"way back when @tkurki proposed the delta format I assumed the context would be the longest common root of the values sent, since thats the most efficient format, eg why duplicate `navigation.`for every nav value, better to have the context `vessels.urn:xxxxxxxx.navigation`. But it causes complexity in parsing, and we decided simplicity was more valuable. So we always use `vessels.urn:xxxxxx`. We have discussed dropping the `vessels.` part as thats always the same.\r\n\r\nMaking context into an object, with attributes `vessel` is a good solution, but breaks the current format. By also including a `path` attribute we could get some efficiency gains by not duplicating common paths.\r\n\r\nBut the use cases can be fulfilled with the current format, without major issues and we are very close to V1 release.\r\n\r\nI favour this change, but we need to include it in a later version as its not critical to V1. "}}
{"actor":{"display_login":"rob42"}}