{"action":"created","issue":{"url":"https://api.github.com/repos/stan-dev/math/issues/217","labels_url":"https://api.github.com/repos/stan-dev/math/issues/217/labels{/name}","comments_url":"https://api.github.com/repos/stan-dev/math/issues/217/comments","events_url":"https://api.github.com/repos/stan-dev/math/issues/217/events","html_url":"https://github.com/stan-dev/math/pull/217","id":121614422,"number":217,"title":"Feature/issue 175 add cvode","user":{"login":"betanalpha","id":1527190,"avatar_url":"https://avatars.githubusercontent.com/u/1527190?v=3","gravatar_id":"","url":"https://api.github.com/users/betanalpha","html_url":"https://github.com/betanalpha","followers_url":"https://api.github.com/users/betanalpha/followers","following_url":"https://api.github.com/users/betanalpha/following{/other_user}","gists_url":"https://api.github.com/users/betanalpha/gists{/gist_id}","starred_url":"https://api.github.com/users/betanalpha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/betanalpha/subscriptions","organizations_url":"https://api.github.com/users/betanalpha/orgs","repos_url":"https://api.github.com/users/betanalpha/repos","events_url":"https://api.github.com/users/betanalpha/events{/privacy}","received_events_url":"https://api.github.com/users/betanalpha/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-12-11T01:19:44Z","updated_at":"2015-12-11T21:17:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/stan-dev/math/pulls/217","html_url":"https://github.com/stan-dev/math/pull/217","diff_url":"https://github.com/stan-dev/math/pull/217.diff","patch_url":"https://github.com/stan-dev/math/pull/217.patch"},"body":"#### Summary:\r\nExposes a stiff CVODE solver using a diagonal Jacobian approximation as the ```integrate_ode_cvode''' function.  The test makefile compiles CVODE into a shared library from source to allow the function to be tested, and the compilation has been tested on OS X and Windows.  Addresses Issue #175 and Issue #52.\r\n\r\n#### Intended Effect:\r\nIntroduces a new ODE integrator capable of handling stiff systems.\r\n\r\n#### How to Verify:\r\ntest/unit/math/prim/arr/functor/coupled_ode_system_cvode_test.cpp\ttest/unit/math/prim/arr/functor/integrate_ode_cvode_test.cpp\r\ntest/unit/math/rev/arr/functor/coupled_ode_system_cvode_test.cpp\ttest/unit/math/rev/arr/functor/integrate_ode_cvode_test.cpp\r\ntest/unit/math/rev/arr/functor/integrate_ode_cvode_grad_test.cpp\ttest/unit/math/rev/arr/functor/util_cvode.hpp\r\n\r\n#### Side Effects\r\nMakefile now compiles CVODE from source and installs a shared library in lib/cvode_2.8.2/.\r\n\r\n#### Documentation:\r\nNone\r\n\r\n#### Reviewer Suggestions:\r\n\r\n@syclik @bob-carpenter "},"comment":{"url":"https://api.github.com/repos/stan-dev/math/issues/comments/164049817","html_url":"https://github.com/stan-dev/math/pull/217#issuecomment-164049817","issue_url":"https://api.github.com/repos/stan-dev/math/issues/217","id":164049817,"user":{"login":"betanalpha","id":1527190,"avatar_url":"https://avatars.githubusercontent.com/u/1527190?v=3","gravatar_id":"","url":"https://api.github.com/users/betanalpha","html_url":"https://github.com/betanalpha","followers_url":"https://api.github.com/users/betanalpha/followers","following_url":"https://api.github.com/users/betanalpha/following{/other_user}","gists_url":"https://api.github.com/users/betanalpha/gists{/gist_id}","starred_url":"https://api.github.com/users/betanalpha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/betanalpha/subscriptions","organizations_url":"https://api.github.com/users/betanalpha/orgs","repos_url":"https://api.github.com/users/betanalpha/repos","events_url":"https://api.github.com/users/betanalpha/events{/privacy}","received_events_url":"https://api.github.com/users/betanalpha/received_events","type":"User","site_admin":false},"created_at":"2015-12-11T21:17:53Z","updated_at":"2015-12-11T21:17:53Z","body":"I think it has to be long for the C methods to match, no?\n\n> On Dec 11, 2015, at 16:11, Daniel Lee <notifications@github.com> wrote:\n> \n> I'm working on it. I was trying out size_t. \n> \n> > On Dec 11, 2015, at 4:09 PM, Michael Betancourt <notifications@github.com> wrote:\n> > \n> > cpplint is complaining about the use of long int'' but a C++ equivalent likeint_least32_t'' won't compile as it doesn't match the necessary C callbacks. Should we just block the cpplint warnings here?\n> > \n> > —\n> > Reply to this email directly or view it on GitHub.\n> > \n> —\n> Reply to this email directly or view it on GitHub.\n> \n"}}