{"action":"created","issue":{"url":"https://api.github.com/repos/Unidata/netcdf-c/issues/120","labels_url":"https://api.github.com/repos/Unidata/netcdf-c/issues/120/labels{/name}","comments_url":"https://api.github.com/repos/Unidata/netcdf-c/issues/120/comments","events_url":"https://api.github.com/repos/Unidata/netcdf-c/issues/120/events","html_url":"https://github.com/Unidata/netcdf-c/pull/120","id":92243227,"number":120,"title":"Remove error check for NC_INT64 type","user":{"login":"gsjaardema","id":897294,"avatar_url":"https://avatars.githubusercontent.com/u/897294?v=3","gravatar_id":"","url":"https://api.github.com/users/gsjaardema","html_url":"https://github.com/gsjaardema","followers_url":"https://api.github.com/users/gsjaardema/followers","following_url":"https://api.github.com/users/gsjaardema/following{/other_user}","gists_url":"https://api.github.com/users/gsjaardema/gists{/gist_id}","starred_url":"https://api.github.com/users/gsjaardema/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gsjaardema/subscriptions","organizations_url":"https://api.github.com/users/gsjaardema/orgs","repos_url":"https://api.github.com/users/gsjaardema/repos","events_url":"https://api.github.com/users/gsjaardema/events{/privacy}","received_events_url":"https://api.github.com/users/gsjaardema/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-01T02:02:53Z","updated_at":"2015-07-06T20:43:23Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Unidata/netcdf-c/pulls/120","html_url":"https://github.com/Unidata/netcdf-c/pull/120","diff_url":"https://github.com/Unidata/netcdf-c/pull/120.diff","patch_url":"https://github.com/Unidata/netcdf-c/pull/120.patch"},"body":"There is a case that handles a memtype of NC_INT64, so the code\r\nshould not error out if that memtype is passed in.  This type is also handled in the other file types, so not sure why it should\r\ncause invalid values error in parallel-netcdf mode."},"comment":{"url":"https://api.github.com/repos/Unidata/netcdf-c/issues/comments/118992897","html_url":"https://github.com/Unidata/netcdf-c/pull/120#issuecomment-118992897","issue_url":"https://api.github.com/repos/Unidata/netcdf-c/issues/120","id":118992897,"user":{"login":"DennisHeimbigner","id":905179,"avatar_url":"https://avatars.githubusercontent.com/u/905179?v=3","gravatar_id":"","url":"https://api.github.com/users/DennisHeimbigner","html_url":"https://github.com/DennisHeimbigner","followers_url":"https://api.github.com/users/DennisHeimbigner/followers","following_url":"https://api.github.com/users/DennisHeimbigner/following{/other_user}","gists_url":"https://api.github.com/users/DennisHeimbigner/gists{/gist_id}","starred_url":"https://api.github.com/users/DennisHeimbigner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DennisHeimbigner/subscriptions","organizations_url":"https://api.github.com/users/DennisHeimbigner/orgs","repos_url":"https://api.github.com/users/DennisHeimbigner/repos","events_url":"https://api.github.com/users/DennisHeimbigner/events{/privacy}","received_events_url":"https://api.github.com/users/DennisHeimbigner/received_events","type":"User","site_admin":false},"created_at":"2015-07-06T20:43:23Z","updated_at":"2015-07-06T20:43:23Z","body":"See e-support \tDMD-262410\t\r\nAs noted there.\r\n\tPosted on: 20150706.1433 MDT\t\r\nI am not sure why that check is there. It was copied over\r\nfrom original code by Ed Hartnett. Looking at the pnetcdf\r\ndocumentation, I ran across this:\r\n\r\n> Starting from 1.3.0, PnetCDF supports  CDF-5 file format:\r\n> adding unsigned and 64-bit integer data types and ...\r\n\r\nIt is possible that this is the source of that check.\r\nIn any case, it seems like the check is no longer needed\r\nand per the pull request, should be removed."}}