{"action":"created","issue":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/3815","repository_url":"https://api.github.com/repos/RobotLocomotion/drake","labels_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/3815/labels{/name}","comments_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/3815/comments","events_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/3815/events","html_url":"https://github.com/RobotLocomotion/drake/issues/3815","id":183245076,"number":3815,"title":"convert textbook to use c++ examples","user":{"login":"RussTedrake","id":6442292,"avatar_url":"https://avatars.githubusercontent.com/u/6442292?v=3","gravatar_id":"","url":"https://api.github.com/users/RussTedrake","html_url":"https://github.com/RussTedrake","followers_url":"https://api.github.com/users/RussTedrake/followers","following_url":"https://api.github.com/users/RussTedrake/following{/other_user}","gists_url":"https://api.github.com/users/RussTedrake/gists{/gist_id}","starred_url":"https://api.github.com/users/RussTedrake/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RussTedrake/subscriptions","organizations_url":"https://api.github.com/users/RussTedrake/orgs","repos_url":"https://api.github.com/users/RussTedrake/repos","events_url":"https://api.github.com/users/RussTedrake/events{/privacy}","received_events_url":"https://api.github.com/users/RussTedrake/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RobotLocomotion/drake/labels/priority:%20low","name":"priority: low","color":"e99695"},{"url":"https://api.github.com/repos/RobotLocomotion/drake/labels/team:%20software%20core","name":"team: software core","color":"0052cc"},{"url":"https://api.github.com/repos/RobotLocomotion/drake/labels/type:%20documentation","name":"type: documentation","color":"009800"}],"state":"open","locked":false,"assignee":{"login":"RussTedrake","id":6442292,"avatar_url":"https://avatars.githubusercontent.com/u/6442292?v=3","gravatar_id":"","url":"https://api.github.com/users/RussTedrake","html_url":"https://github.com/RussTedrake","followers_url":"https://api.github.com/users/RussTedrake/followers","following_url":"https://api.github.com/users/RussTedrake/following{/other_user}","gists_url":"https://api.github.com/users/RussTedrake/gists{/gist_id}","starred_url":"https://api.github.com/users/RussTedrake/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RussTedrake/subscriptions","organizations_url":"https://api.github.com/users/RussTedrake/orgs","repos_url":"https://api.github.com/users/RussTedrake/repos","events_url":"https://api.github.com/users/RussTedrake/events{/privacy}","received_events_url":"https://api.github.com/users/RussTedrake/received_events","type":"User","site_admin":false},"assignees":[{"login":"RussTedrake","id":6442292,"avatar_url":"https://avatars.githubusercontent.com/u/6442292?v=3","gravatar_id":"","url":"https://api.github.com/users/RussTedrake","html_url":"https://github.com/RussTedrake","followers_url":"https://api.github.com/users/RussTedrake/followers","following_url":"https://api.github.com/users/RussTedrake/following{/other_user}","gists_url":"https://api.github.com/users/RussTedrake/gists{/gist_id}","starred_url":"https://api.github.com/users/RussTedrake/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RussTedrake/subscriptions","organizations_url":"https://api.github.com/users/RussTedrake/orgs","repos_url":"https://api.github.com/users/RussTedrake/repos","events_url":"https://api.github.com/users/RussTedrake/events{/privacy}","received_events_url":"https://api.github.com/users/RussTedrake/received_events","type":"User","site_admin":false}],"milestone":null,"comments":12,"created_at":"2016-10-16T02:33:14Z","updated_at":"2016-10-17T18:52:57Z","closed_at":null,"body":"I am working on converting the textbook / drake tutorial to use the new c++ classes.  To make it work well, I think I need to change the relationship between the textbook and the drake repo.  \r\n\r\nProposal:\r\n  - textbook moves out of `drake/doc` and into `externals/textbook`\r\n  - textbook compiles after drake, and contains it's own `src/` directory for the examples which are still compiled/run as part of drake's CI.\r\n  - the simplest examples (like `simple_continuous_time_system.cc`) move out of `drake/examples` and into `textbook/src`\r\n\r\nMy questions/observations/concerns:\r\n  - this is very similar to the proposed model for director (build as an external after drake)\r\n  - changes to drake that would break the textbook examples are a potential pain point (presumably for director, too).  will require PR'ing to the textbook, and updating the SHA.  I can try to keep these dependencies very minimal to begin with. \r\n  - textbook examples won't benefit from the same code review without me asking (but I do intend to ask).  they also need not have the same linters/etc enforced, and will not appear in the doxygen.\r\n\r\nOtherwise, I actually think this is a cleaner (more explicit) relationship between the two.  Soliciting feedback now in case there are objections, or if people foresee an issue that I've missed.\r\n\r\n@jamiesnape , @david-german-tri, @jwnimmer-tri -- would value your feedback in particular."},"comment":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/comments/254298675","html_url":"https://github.com/RobotLocomotion/drake/issues/3815#issuecomment-254298675","issue_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/3815","id":254298675,"user":{"login":"RussTedrake","id":6442292,"avatar_url":"https://avatars.githubusercontent.com/u/6442292?v=3","gravatar_id":"","url":"https://api.github.com/users/RussTedrake","html_url":"https://github.com/RussTedrake","followers_url":"https://api.github.com/users/RussTedrake/followers","following_url":"https://api.github.com/users/RussTedrake/following{/other_user}","gists_url":"https://api.github.com/users/RussTedrake/gists{/gist_id}","starred_url":"https://api.github.com/users/RussTedrake/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RussTedrake/subscriptions","organizations_url":"https://api.github.com/users/RussTedrake/orgs","repos_url":"https://api.github.com/users/RussTedrake/repos","events_url":"https://api.github.com/users/RussTedrake/events{/privacy}","received_events_url":"https://api.github.com/users/RussTedrake/received_events","type":"User","site_admin":false},"created_at":"2016-10-17T18:52:57Z","updated_at":"2016-10-17T18:52:57Z","body":"it's worth noting that the current state is that we have the textbook in a separate repo, owning its own example code, which could break on an API change to drake.  that has not happened yet because it was only matlab example code.  the fact that it gets loaded into `drake/doc/textbook` avoids the need for `find_package(drake)`, but is otherwise almost irrelevant.\r\n\r\ni do think that the textbook text belongs in a different repo -- it is logically separable and I will need to commit text relatively often that the drake developers probably don't care to review.  it's also convenient to be able to grab the textbook without all of drake.  "}}
{"actor":{"display_login":"RussTedrake"}}