{"action":"created","issue":{"url":"https://api.github.com/repos/nimble-dev/nimble/issues/433","repository_url":"https://api.github.com/repos/nimble-dev/nimble","labels_url":"https://api.github.com/repos/nimble-dev/nimble/issues/433/labels{/name}","comments_url":"https://api.github.com/repos/nimble-dev/nimble/issues/433/comments","events_url":"https://api.github.com/repos/nimble-dev/nimble/issues/433/events","html_url":"https://github.com/nimble-dev/nimble/pull/433","id":235966499,"number":433,"title":"Omnibus","user":{"login":"perrydv","id":8012337,"avatar_url":"https://avatars0.githubusercontent.com/u/8012337?v=3","gravatar_id":"","url":"https://api.github.com/users/perrydv","html_url":"https://github.com/perrydv","followers_url":"https://api.github.com/users/perrydv/followers","following_url":"https://api.github.com/users/perrydv/following{/other_user}","gists_url":"https://api.github.com/users/perrydv/gists{/gist_id}","starred_url":"https://api.github.com/users/perrydv/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/perrydv/subscriptions","organizations_url":"https://api.github.com/users/perrydv/orgs","repos_url":"https://api.github.com/users/perrydv/repos","events_url":"https://api.github.com/users/perrydv/events{/privacy}","received_events_url":"https://api.github.com/users/perrydv/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2017-06-14T18:17:27Z","updated_at":"2017-06-21T19:02:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nimble-dev/nimble/pulls/433","html_url":"https://github.com/nimble-dev/nimble/pull/433","diff_url":"https://github.com/nimble-dev/nimble/pull/433.diff","patch_url":"https://github.com/nimble-dev/nimble/pull/433.patch"},"body":"This PR fixes six old and/or minor issues:\r\n\r\n1. Issue #378: I made options(error = recover) work for the main case of interest: size processing called from `RCfunProcessing::process`.  Proposed solution is to defer to R’s native handling if options(‘error’) is non-null, and otherwise to use our existing  `tryCatch` clause.\r\n\r\n2. Issue #374: I removed the `MCMC_*` files from the repo above the R package level.  According to the issue, @paciorek has these files saved separately for when they are needed but they do not need to live in the repo.\r\n\r\n3. Issue #365: support for keyword `next` was added. A test was added to `test-misc.R`.\r\n\r\n4. Issue #314: `NaN` is now successfully written to C++ from R via a `nimble_NaN()` function that attempts to use `std::numeric_limits::quiet_NaN()` when available and otherwise computes a NaN as 0./0.  A test was added to `test-misc.R`.\r\n\r\n5. Issue #302: dots in R variable names in the DSL (e.g. `a.b`) are now handled by processing through our heavily used utility `Rname2CppName`. This occurs at the final step of writing C++ code.  The mangled result is `a_dot_b`.\r\n\r\n6. Issue #92: This old issue was to support `pi` in the DSL.  Brief discussion led to a sticking point and it was left to go stale.  In R, `pi` exists as a constant that can be overwritten by the user.  Proposed solution to yield similar compiled behavior is that if a variable `pi` is used, it is always initialized to the natural constant `pi`, after which it behaves like any other local variable and hence can be overwritten.  Behavior will be limited by static typing: if used first on RHS, it will be a scalar double and can only be over-written as such.  Three tests were added to `test-misc.R`.\r\n\r\n**Note that `pi` still does not work in model code.** We can open that as a separate issue if desired: it involves model processing and was not handled as part of Issue #92.  "},"comment":{"url":"https://api.github.com/repos/nimble-dev/nimble/issues/comments/310174681","html_url":"https://github.com/nimble-dev/nimble/pull/433#issuecomment-310174681","issue_url":"https://api.github.com/repos/nimble-dev/nimble/issues/433","id":310174681,"user":{"login":"paciorek","id":2501598,"avatar_url":"https://avatars1.githubusercontent.com/u/2501598?v=3","gravatar_id":"","url":"https://api.github.com/users/paciorek","html_url":"https://github.com/paciorek","followers_url":"https://api.github.com/users/paciorek/followers","following_url":"https://api.github.com/users/paciorek/following{/other_user}","gists_url":"https://api.github.com/users/paciorek/gists{/gist_id}","starred_url":"https://api.github.com/users/paciorek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/paciorek/subscriptions","organizations_url":"https://api.github.com/users/paciorek/orgs","repos_url":"https://api.github.com/users/paciorek/repos","events_url":"https://api.github.com/users/paciorek/events{/privacy}","received_events_url":"https://api.github.com/users/paciorek/received_events","type":"User","site_admin":false},"created_at":"2017-06-21T19:02:22Z","updated_at":"2017-06-21T19:02:22Z","body":"No concerns on my part; didn't look very deeply."}}
{"actor":{"display_login":"paciorek"}}