{"action":"created","issue":{"url":"https://api.github.com/repos/ropensci/cleanEHR/issues/165","repository_url":"https://api.github.com/repos/ropensci/cleanEHR","labels_url":"https://api.github.com/repos/ropensci/cleanEHR/issues/165/labels{/name}","comments_url":"https://api.github.com/repos/ropensci/cleanEHR/issues/165/comments","events_url":"https://api.github.com/repos/ropensci/cleanEHR/issues/165/events","html_url":"https://github.com/ropensci/cleanEHR/pull/165","id":345157192,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA0Mzc4NTk3","number":165,"title":"Time bug fix","user":{"login":"DocEd","id":17110713,"node_id":"MDQ6VXNlcjE3MTEwNzEz","avatar_url":"https://avatars2.githubusercontent.com/u/17110713?v=4","gravatar_id":"","url":"https://api.github.com/users/DocEd","html_url":"https://github.com/DocEd","followers_url":"https://api.github.com/users/DocEd/followers","following_url":"https://api.github.com/users/DocEd/following{/other_user}","gists_url":"https://api.github.com/users/DocEd/gists{/gist_id}","starred_url":"https://api.github.com/users/DocEd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DocEd/subscriptions","organizations_url":"https://api.github.com/users/DocEd/orgs","repos_url":"https://api.github.com/users/DocEd/repos","events_url":"https://api.github.com/users/DocEd/events{/privacy}","received_events_url":"https://api.github.com/users/DocEd/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-07-27T09:32:15Z","updated_at":"2018-07-27T11:02:37Z","closed_at":null,"author_association":"COLLABORATOR","pull_request":{"url":"https://api.github.com/repos/ropensci/cleanEHR/pulls/165","html_url":"https://github.com/ropensci/cleanEHR/pull/165","diff_url":"https://github.com/ropensci/cleanEHR/pull/165.diff","patch_url":"https://github.com/ropensci/cleanEHR/pull/165.patch"},"body":"This segment of the code caused a problem with allocation of memory.\r\nIt looks like `findMaxTime()` was the culprit returning a very high number (possibly the number of hours since 1900??)\r\n\r\nThis modification works, but may effect legacy `ccRecords` that were created outside the Postgres pipeline. I'm not very familiar with that aspect, so this needs review in the context of preserving legacy features.\r\n\r\nI have removed reference to `is.na(tadm)` because this field is **always** present in the new system (A record can't exist without an admission time)\r\n\r\nMany thanks in advance for input."},"comment":{"url":"https://api.github.com/repos/ropensci/cleanEHR/issues/comments/408386473","html_url":"https://github.com/ropensci/cleanEHR/pull/165#issuecomment-408386473","issue_url":"https://api.github.com/repos/ropensci/cleanEHR/issues/165","id":408386473,"node_id":"MDEyOklzc3VlQ29tbWVudDQwODM4NjQ3Mw==","user":{"login":"DocEd","id":17110713,"node_id":"MDQ6VXNlcjE3MTEwNzEz","avatar_url":"https://avatars2.githubusercontent.com/u/17110713?v=4","gravatar_id":"","url":"https://api.github.com/users/DocEd","html_url":"https://github.com/DocEd","followers_url":"https://api.github.com/users/DocEd/followers","following_url":"https://api.github.com/users/DocEd/following{/other_user}","gists_url":"https://api.github.com/users/DocEd/gists{/gist_id}","starred_url":"https://api.github.com/users/DocEd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DocEd/subscriptions","organizations_url":"https://api.github.com/users/DocEd/orgs","repos_url":"https://api.github.com/users/DocEd/repos","events_url":"https://api.github.com/users/DocEd/events{/privacy}","received_events_url":"https://api.github.com/users/DocEd/received_events","type":"User","site_admin":false},"created_at":"2018-07-27T11:02:37Z","updated_at":"2018-07-27T11:02:37Z","author_association":"COLLABORATOR","body":"Not sure. I'll check with Roma a little later on today"}}
{"actor":{"display_login":"DocEd"}}