{"action":"created","issue":{"url":"https://api.github.com/repos/andrewpbray/infer/issues/36","repository_url":"https://api.github.com/repos/andrewpbray/infer","labels_url":"https://api.github.com/repos/andrewpbray/infer/issues/36/labels{/name}","comments_url":"https://api.github.com/repos/andrewpbray/infer/issues/36/comments","events_url":"https://api.github.com/repos/andrewpbray/infer/issues/36/events","html_url":"https://github.com/andrewpbray/infer/issues/36","id":244353799,"number":36,"title":"Handling of NAs","user":{"login":"mine-cetinkaya-rundel","id":5965649,"avatar_url":"https://avatars0.githubusercontent.com/u/5965649?v=4","gravatar_id":"","url":"https://api.github.com/users/mine-cetinkaya-rundel","html_url":"https://github.com/mine-cetinkaya-rundel","followers_url":"https://api.github.com/users/mine-cetinkaya-rundel/followers","following_url":"https://api.github.com/users/mine-cetinkaya-rundel/following{/other_user}","gists_url":"https://api.github.com/users/mine-cetinkaya-rundel/gists{/gist_id}","starred_url":"https://api.github.com/users/mine-cetinkaya-rundel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mine-cetinkaya-rundel/subscriptions","organizations_url":"https://api.github.com/users/mine-cetinkaya-rundel/orgs","repos_url":"https://api.github.com/users/mine-cetinkaya-rundel/repos","events_url":"https://api.github.com/users/mine-cetinkaya-rundel/events{/privacy}","received_events_url":"https://api.github.com/users/mine-cetinkaya-rundel/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-07-20T13:00:05Z","updated_at":"2017-07-20T15:08:56Z","closed_at":null,"body":"Currently, if the input data has an NA, no warnings are shown but the resulting simulated statistics vector can have some NAs since `calculate` doesn't specify to handle NAs when taking mean, median, etc. One option is to specify that in calculate. The other option is to make the user get rid of NAs first in the exploratory stage, and then feed the data into the `infer` steps. Former is a bit more user friendly, but hides an important consideration. Latter can be tricky for students when doing inference for two variables since they would need to remove pairs of NAs from both variables. Thoughts?"},"comment":{"url":"https://api.github.com/repos/andrewpbray/infer/issues/comments/316733513","html_url":"https://github.com/andrewpbray/infer/issues/36#issuecomment-316733513","issue_url":"https://api.github.com/repos/andrewpbray/infer/issues/36","id":316733513,"user":{"login":"ismayc","id":9215614,"avatar_url":"https://avatars3.githubusercontent.com/u/9215614?v=4","gravatar_id":"","url":"https://api.github.com/users/ismayc","html_url":"https://github.com/ismayc","followers_url":"https://api.github.com/users/ismayc/followers","following_url":"https://api.github.com/users/ismayc/following{/other_user}","gists_url":"https://api.github.com/users/ismayc/gists{/gist_id}","starred_url":"https://api.github.com/users/ismayc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ismayc/subscriptions","organizations_url":"https://api.github.com/users/ismayc/orgs","repos_url":"https://api.github.com/users/ismayc/repos","events_url":"https://api.github.com/users/ismayc/events{/privacy}","received_events_url":"https://api.github.com/users/ismayc/received_events","type":"User","site_admin":false},"created_at":"2017-07-20T15:08:56Z","updated_at":"2017-07-20T15:08:56Z","body":"I'm inclined to have students do the removal of NAs themselves in the exploratory stage instead of doing it inside `calculate` since I think it is important for students to understand their data before they throw it into a pipeline like this."}}
{"actor":{"display_login":"ismayc"}}