{"action":"created","issue":{"url":"https://api.github.com/repos/all-of-us/raw-data-repository/issues/517","repository_url":"https://api.github.com/repos/all-of-us/raw-data-repository","labels_url":"https://api.github.com/repos/all-of-us/raw-data-repository/issues/517/labels{/name}","comments_url":"https://api.github.com/repos/all-of-us/raw-data-repository/issues/517/comments","events_url":"https://api.github.com/repos/all-of-us/raw-data-repository/issues/517/events","html_url":"https://github.com/all-of-us/raw-data-repository/pull/517","id":271089918,"number":517,"title":"Changing measurement specs to match latest physical measurements code…","user":{"login":"danqrodney","id":12536316,"avatar_url":"https://avatars0.githubusercontent.com/u/12536316?v=4","gravatar_id":"","url":"https://api.github.com/users/danqrodney","html_url":"https://github.com/danqrodney","followers_url":"https://api.github.com/users/danqrodney/followers","following_url":"https://api.github.com/users/danqrodney/following{/other_user}","gists_url":"https://api.github.com/users/danqrodney/gists{/gist_id}","starred_url":"https://api.github.com/users/danqrodney/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danqrodney/subscriptions","organizations_url":"https://api.github.com/users/danqrodney/orgs","repos_url":"https://api.github.com/users/danqrodney/repos","events_url":"https://api.github.com/users/danqrodney/events{/privacy}","received_events_url":"https://api.github.com/users/danqrodney/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-11-03T19:15:43Z","updated_at":"2017-11-03T19:22:55Z","closed_at":null,"author_association":"COLLABORATOR","pull_request":{"url":"https://api.github.com/repos/all-of-us/raw-data-repository/pulls/517","html_url":"https://github.com/all-of-us/raw-data-repository/pull/517","diff_url":"https://github.com/all-of-us/raw-data-repository/pull/517.diff","patch_url":"https://github.com/all-of-us/raw-data-repository/pull/517.patch"},"body":"…book\r\n\r\nSee: \r\n\r\nhttps://docs.google.com/spreadsheets/d/1KMPxUMjlQLNK7dO1QMVxDTLJ2N5SksLq2Qdp9MCSOMw/edit#gid=1076265690"},"comment":{"url":"https://api.github.com/repos/all-of-us/raw-data-repository/issues/comments/341802364","html_url":"https://github.com/all-of-us/raw-data-repository/pull/517#issuecomment-341802364","issue_url":"https://api.github.com/repos/all-of-us/raw-data-repository/issues/517","id":341802364,"user":{"login":"danqrodney","id":12536316,"avatar_url":"https://avatars0.githubusercontent.com/u/12536316?v=4","gravatar_id":"","url":"https://api.github.com/users/danqrodney","html_url":"https://github.com/danqrodney","followers_url":"https://api.github.com/users/danqrodney/followers","following_url":"https://api.github.com/users/danqrodney/following{/other_user}","gists_url":"https://api.github.com/users/danqrodney/gists{/gist_id}","starred_url":"https://api.github.com/users/danqrodney/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danqrodney/subscriptions","organizations_url":"https://api.github.com/users/danqrodney/orgs","repos_url":"https://api.github.com/users/danqrodney/repos","events_url":"https://api.github.com/users/danqrodney/events{/privacy}","received_events_url":"https://api.github.com/users/danqrodney/received_events","type":"User","site_admin":false},"created_at":"2017-11-03T19:22:55Z","updated_at":"2017-11-03T19:22:55Z","author_association":"COLLABORATOR","body":"So the original spec was generated, by this code which takes a sample of physical measurements actually in use in prod and calculates their range of values:\r\n\r\nhttps://github.com/all-of-us/raw-data-repository/blob/master/rest-api/tools/validate_or_backfill_measurements.py\r\n\r\nHowever, since then we've updated to a new way of representing physical measurements, that isn't yet reflected in prod (but soon will be.) (Note that the above script won't quite work right with those changes, now that clients are actually going to send PMI-specific and LOINC codes for measurements... at some point we'll need to update the logic it's using.)\r\n\r\nThis PR is updating our spec to match how the data is going to look with the new measurements, so we can generate realisting synthetic data (for use in generating a CDR.)\r\n"}}
{"actor":{"display_login":"danqrodney"}}