{"action":"created","issue":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/7975","repository_url":"https://api.github.com/repos/RobotLocomotion/drake","labels_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/7975/labels{/name}","comments_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/7975/comments","events_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/7975/events","html_url":"https://github.com/RobotLocomotion/drake/pull/7975","id":294986535,"number":7975,"title":"Add SNOPT 7.5 support while retaining RLG support","user":{"login":"jwnimmer-tri","id":17596505,"avatar_url":"https://avatars3.githubusercontent.com/u/17596505?v=4","gravatar_id":"","url":"https://api.github.com/users/jwnimmer-tri","html_url":"https://github.com/jwnimmer-tri","followers_url":"https://api.github.com/users/jwnimmer-tri/followers","following_url":"https://api.github.com/users/jwnimmer-tri/following{/other_user}","gists_url":"https://api.github.com/users/jwnimmer-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/jwnimmer-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jwnimmer-tri/subscriptions","organizations_url":"https://api.github.com/users/jwnimmer-tri/orgs","repos_url":"https://api.github.com/users/jwnimmer-tri/repos","events_url":"https://api.github.com/users/jwnimmer-tri/events{/privacy}","received_events_url":"https://api.github.com/users/jwnimmer-tri/received_events","type":"User","site_admin":false},"labels":[{"id":353617587,"url":"https://api.github.com/repos/RobotLocomotion/drake/labels/status:%20do%20not%20merge","name":"status: do not merge","color":"e67e22","default":false}],"state":"open","locked":false,"assignee":{"login":"ggould-tri","id":17596413,"avatar_url":"https://avatars2.githubusercontent.com/u/17596413?v=4","gravatar_id":"","url":"https://api.github.com/users/ggould-tri","html_url":"https://github.com/ggould-tri","followers_url":"https://api.github.com/users/ggould-tri/followers","following_url":"https://api.github.com/users/ggould-tri/following{/other_user}","gists_url":"https://api.github.com/users/ggould-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/ggould-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ggould-tri/subscriptions","organizations_url":"https://api.github.com/users/ggould-tri/orgs","repos_url":"https://api.github.com/users/ggould-tri/repos","events_url":"https://api.github.com/users/ggould-tri/events{/privacy}","received_events_url":"https://api.github.com/users/ggould-tri/received_events","type":"User","site_admin":false},"assignees":[{"login":"ggould-tri","id":17596413,"avatar_url":"https://avatars2.githubusercontent.com/u/17596413?v=4","gravatar_id":"","url":"https://api.github.com/users/ggould-tri","html_url":"https://github.com/ggould-tri","followers_url":"https://api.github.com/users/ggould-tri/followers","following_url":"https://api.github.com/users/ggould-tri/following{/other_user}","gists_url":"https://api.github.com/users/ggould-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/ggould-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ggould-tri/subscriptions","organizations_url":"https://api.github.com/users/ggould-tri/orgs","repos_url":"https://api.github.com/users/ggould-tri/repos","events_url":"https://api.github.com/users/ggould-tri/events{/privacy}","received_events_url":"https://api.github.com/users/ggould-tri/received_events","type":"User","site_admin":false}],"milestone":null,"comments":2,"created_at":"2018-02-07T02:37:54Z","updated_at":"2018-02-08T15:38:50Z","closed_at":null,"author_association":"COLLABORATOR","pull_request":{"url":"https://api.github.com/repos/RobotLocomotion/drake/pulls/7975","html_url":"https://github.com/RobotLocomotion/drake/pull/7975","diff_url":"https://github.com/RobotLocomotion/drake/pull/7975.diff","patch_url":"https://github.com/RobotLocomotion/drake/pull/7975.patch"},"body":"This PR amends the BUILD rules and C++ code to allow for SNOPT 7.5.  If an external workspace were to call our rule with a different git remote and commit that pointed at SNOPT 7.5, this would build and pass `snopt_solver_test` cases.\r\n\r\nRelates to #1647 but does not fix it yet, since the default is still SNOPT 7.2.\r\n\r\nDo not merge until:\r\n- [ ] tested on macOS.\r\n- [x] agreed testing plan (or at least forecast) for SNOPT 7.5 in CI.\r\n  - Discussed in #1647.\r\n  - I have manually tested the `package.BUILD.bazel` in this PR against SNOPT 7.5, using the archive selector prototyped at https://github.com/jwnimmer-tri/drake/tree/snopt, running `test //solvers:snopt_solver_test`.\r\n\r\n<!-- Reviewable:start -->\r\n---\r\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/robotlocomotion/drake/7975)\r\n<!-- Reviewable:end -->\r\n"},"comment":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/comments/364150443","html_url":"https://github.com/RobotLocomotion/drake/pull/7975#issuecomment-364150443","issue_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/7975","id":364150443,"user":{"login":"jwnimmer-tri","id":17596505,"avatar_url":"https://avatars3.githubusercontent.com/u/17596505?v=4","gravatar_id":"","url":"https://api.github.com/users/jwnimmer-tri","html_url":"https://github.com/jwnimmer-tri","followers_url":"https://api.github.com/users/jwnimmer-tri/followers","following_url":"https://api.github.com/users/jwnimmer-tri/following{/other_user}","gists_url":"https://api.github.com/users/jwnimmer-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/jwnimmer-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jwnimmer-tri/subscriptions","organizations_url":"https://api.github.com/users/jwnimmer-tri/orgs","repos_url":"https://api.github.com/users/jwnimmer-tri/repos","events_url":"https://api.github.com/users/jwnimmer-tri/events{/privacy}","received_events_url":"https://api.github.com/users/jwnimmer-tri/received_events","type":"User","site_admin":false},"created_at":"2018-02-08T15:38:50Z","updated_at":"2018-02-08T15:38:50Z","author_association":"COLLABORATOR","body":"\n\n\n\nReview status: 1 of 2 files reviewed at latest revision, 1 unresolved discussion.\n\n---\n\n*[tools/workspace/snopt/package.BUILD.bazel, line 60 at r1](https://reviewable.io:443/reviews/robotlocomotion/drake/7975#-L4pk4tN48lF8bunciQ0:-L4q3OvC4SmyCW5oa6TX:b-896fix) ([raw file](https://github.com/robotlocomotion/drake/blob/ab5292d9d363b255208596f831c433a34b10347f/tools/workspace/snopt/package.BUILD.bazel#L60)):*\n<details><summary><i>Previously, ggould-tri wrote…</i></summary><blockquote>\n\nBTW -- this may be a bit cryptic of the the file is not found, since it will give a bounds error on the `[0]`.  Consider `(glob(...) or fail(...))[0]` to get a human-readable indication of the actual failure circumstance?\n</blockquote></details>\n\nDone.\n\n---\n\n\n*Comments from [Reviewable](https://reviewable.io:443/reviews/robotlocomotion/drake/7975)*\n<!-- Sent from Reviewable.io -->\n"}}
{"actor":{"display_login":"jwnimmer-tri"}}