{"action":"created","issue":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2970","repository_url":"https://api.github.com/repos/RobotLocomotion/drake","labels_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2970/labels{/name}","comments_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2970/comments","events_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2970/events","html_url":"https://github.com/RobotLocomotion/drake/pull/2970","id":168172945,"number":2970,"title":"Add a continuous integrator to the System2 primitive repertoire.","user":{"login":"david-german-tri","id":17437069,"avatar_url":"https://avatars.githubusercontent.com/u/17437069?v=3","gravatar_id":"","url":"https://api.github.com/users/david-german-tri","html_url":"https://github.com/david-german-tri","followers_url":"https://api.github.com/users/david-german-tri/followers","following_url":"https://api.github.com/users/david-german-tri/following{/other_user}","gists_url":"https://api.github.com/users/david-german-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/david-german-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/david-german-tri/subscriptions","organizations_url":"https://api.github.com/users/david-german-tri/orgs","repos_url":"https://api.github.com/users/david-german-tri/repos","events_url":"https://api.github.com/users/david-german-tri/events{/privacy}","received_events_url":"https://api.github.com/users/david-german-tri/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"sherm1","id":4088016,"avatar_url":"https://avatars.githubusercontent.com/u/4088016?v=3","gravatar_id":"","url":"https://api.github.com/users/sherm1","html_url":"https://github.com/sherm1","followers_url":"https://api.github.com/users/sherm1/followers","following_url":"https://api.github.com/users/sherm1/following{/other_user}","gists_url":"https://api.github.com/users/sherm1/gists{/gist_id}","starred_url":"https://api.github.com/users/sherm1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sherm1/subscriptions","organizations_url":"https://api.github.com/users/sherm1/orgs","repos_url":"https://api.github.com/users/sherm1/repos","events_url":"https://api.github.com/users/sherm1/events{/privacy}","received_events_url":"https://api.github.com/users/sherm1/received_events","type":"User","site_admin":false},"assignees":[{"login":"sherm1","id":4088016,"avatar_url":"https://avatars.githubusercontent.com/u/4088016?v=3","gravatar_id":"","url":"https://api.github.com/users/sherm1","html_url":"https://github.com/sherm1","followers_url":"https://api.github.com/users/sherm1/followers","following_url":"https://api.github.com/users/sherm1/following{/other_user}","gists_url":"https://api.github.com/users/sherm1/gists{/gist_id}","starred_url":"https://api.github.com/users/sherm1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sherm1/subscriptions","organizations_url":"https://api.github.com/users/sherm1/orgs","repos_url":"https://api.github.com/users/sherm1/repos","events_url":"https://api.github.com/users/sherm1/events{/privacy}","received_events_url":"https://api.github.com/users/sherm1/received_events","type":"User","site_admin":false},{"login":"amcastro-tri","id":17601461,"avatar_url":"https://avatars.githubusercontent.com/u/17601461?v=3","gravatar_id":"","url":"https://api.github.com/users/amcastro-tri","html_url":"https://github.com/amcastro-tri","followers_url":"https://api.github.com/users/amcastro-tri/followers","following_url":"https://api.github.com/users/amcastro-tri/following{/other_user}","gists_url":"https://api.github.com/users/amcastro-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/amcastro-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amcastro-tri/subscriptions","organizations_url":"https://api.github.com/users/amcastro-tri/orgs","repos_url":"https://api.github.com/users/amcastro-tri/repos","events_url":"https://api.github.com/users/amcastro-tri/events{/privacy}","received_events_url":"https://api.github.com/users/amcastro-tri/received_events","type":"User","site_admin":false}],"milestone":null,"comments":23,"created_at":"2016-07-28T19:09:32Z","updated_at":"2016-08-03T17:41:59Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RobotLocomotion/drake/pulls/2970","html_url":"https://github.com/RobotLocomotion/drake/pull/2970","diff_url":"https://github.com/RobotLocomotion/drake/pull/2970.diff","patch_url":"https://github.com/RobotLocomotion/drake/pull/2970.patch"},"body":"This was a recent topic of conversation, and is useful for unit testing diagram state.  @amcastro-tri and @sherm1 for feature review.\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/robotlocomotion/drake/2970)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/comments/237311383","html_url":"https://github.com/RobotLocomotion/drake/pull/2970#issuecomment-237311383","issue_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2970","id":237311383,"user":{"login":"david-german-tri","id":17437069,"avatar_url":"https://avatars.githubusercontent.com/u/17437069?v=3","gravatar_id":"","url":"https://api.github.com/users/david-german-tri","html_url":"https://github.com/david-german-tri","followers_url":"https://api.github.com/users/david-german-tri/followers","following_url":"https://api.github.com/users/david-german-tri/following{/other_user}","gists_url":"https://api.github.com/users/david-german-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/david-german-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/david-german-tri/subscriptions","organizations_url":"https://api.github.com/users/david-german-tri/orgs","repos_url":"https://api.github.com/users/david-german-tri/repos","events_url":"https://api.github.com/users/david-german-tri/events{/privacy}","received_events_url":"https://api.github.com/users/david-german-tri/received_events","type":"User","site_admin":false},"created_at":"2016-08-03T17:41:59Z","updated_at":"2016-08-03T17:41:59Z","body":"\n\n\n\nReview status: 6 of 7 files reviewed at latest revision, 7 unresolved discussions.\n\n---\n\n*[drake/systems/framework/primitives/integrator.h, line 32 \\[r1\\]](https://reviewable.io:443/reviews/robotlocomotion/drake/2970#-KNrG4XWFHkTplOoCEJi:-KOGSzQd1a_jedOz_dEm:-1742952948) ([raw file](https://github.com/robotlocomotion/drake/blob/ee3773116d5ebb744e462fae7b371f3ebba90bf8/drake/systems/framework/primitives/integrator.h#L32)):*\n<details><summary><i>Previously, sherm1 (Michael Sherman) wrote…</i></summary>\n> > Did I say something that suggested otherwise?\n> \n> No, my bad. I misread your comment about where the initial conditions should get calculated. If there is user code around to calculate the ICs, that works. Having the ICs wired into the diagram in Simulink fashion allows the System designer to embed the IC calculation as part of the System and then enables the Solver to perform that computation prior to beginning a simulation. Otherwise the user of a System must understand the requirements for initializing that System. Those requirements may be complicated and require a higher level of skill than a user would otherwise need to use the System successfully.  We can do without that for now but I think we should assume there will be an IC evaluation phase for a System (where only ICs are calculated). \n</details>\nWell, perhaps.  I'm uncomfortable with that for the same reasons I don't want to treat parameters as inputs: if we need a general-purpose programming language, we already have C++.  It sounds like we agree that's a debate for another day, though.\n\n---\n\n\n*Comments from [Reviewable](https://reviewable.io:443/reviews/robotlocomotion/drake/2970)*\n<!-- Sent from Reviewable.io -->\n"}}
{"actor":{"display_login":"david-german-tri"}}