{"action":"created","issue":{"url":"https://api.github.com/repos/aodn/compliance-checker/issues/12","labels_url":"https://api.github.com/repos/aodn/compliance-checker/issues/12/labels{/name}","comments_url":"https://api.github.com/repos/aodn/compliance-checker/issues/12/comments","events_url":"https://api.github.com/repos/aodn/compliance-checker/issues/12/events","html_url":"https://github.com/aodn/compliance-checker/issues/12","id":92248718,"number":12,"title":"CF : check_missing_data","user":{"login":"lbesnard","id":2382814,"avatar_url":"https://avatars.githubusercontent.com/u/2382814?v=3","gravatar_id":"","url":"https://api.github.com/users/lbesnard","html_url":"https://github.com/lbesnard","followers_url":"https://api.github.com/users/lbesnard/followers","following_url":"https://api.github.com/users/lbesnard/following{/other_user}","gists_url":"https://api.github.com/users/lbesnard/gists{/gist_id}","starred_url":"https://api.github.com/users/lbesnard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lbesnard/subscriptions","organizations_url":"https://api.github.com/users/lbesnard/orgs","repos_url":"https://api.github.com/users/lbesnard/repos","events_url":"https://api.github.com/users/lbesnard/events{/privacy}","received_events_url":"https://api.github.com/users/lbesnard/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/aodn/compliance-checker/labels/file_to_fix","name":"file_to_fix","color":"006b75"},{"url":"https://api.github.com/repos/aodn/compliance-checker/labels/matlab_toolbox","name":"matlab_toolbox","color":"fad8c7"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-07-01T02:30:19Z","updated_at":"2015-07-01T07:33:30Z","closed_at":null,"body":"Running Compliance Checker on the dataset from: /mnt/ebs/wip/ANMN/timeSeries/new_tb/IMOS_ANMN-WA_TZ_20140925T080000Z_WATR50_FV01_WATR50-1409-TR-1060-496_END-20150522T023000Z_C-20150522T092647Z.nc (actually affects most of ITFTIS_FV01 files)\r\n\r\nChecker output:\r\n```\r\n missing_data                   :2:     0/ 1 : The auxillary coordinates\r\n                                                      do not have the same\r\n                                                      missing data locations,\r\n                                                      The coordinate variables\r\n                                                      do not have the same\r\n                                                      missing data locations as\r\n                                                      the auxillary coordinates\r\n\r\n```\r\nwhich means \r\n\r\n``` It is not permitted for auxiliary coordinate variables to have missing values for elements where there is non-missing data.```  if you understand what this means, I wouldn't mind an explanation. See 9.6 chapter\r\n\r\n\r\nThis the case for the  DEPTH AND TEMP variables (in the file mentionned above)\r\n\r\nNcdump output:\r\n```\r\nfloat DEPTH(TIME) ;\r\n                DEPTH:coordinates = \"TIME LATITUDE LONGITUDE NOMINAL_DEPTH DEPTH\" ;\r\n                DEPTH:_FillValue = 999999.f ;\r\n\r\n...\r\nfloat TEMP(TIME) ;\r\n                TEMP:coordinates = \"TIME LATITUDE LONGITUDE NOMINAL_DEPTH DEPTH\" ;\r\n                TEMP:_FillValue = 999999.f ;\r\n```\r\nBoth DEPTH and TEMP have a coordinate attribute equals to 'TIME LATITUDE LONGITUDE NOMINAL_DEPTH '. But all these coordinates are missing the ```_FillValue``` attribute. This seems to be the reason why the checker is failing.\r\n\r\nHowever, DEPTH won't have a coordinate attribute in the future (https://github.com/aodn/compliance-checker/issues/8) so does this solve half of the issue ?\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/aodn/compliance-checker/issues/comments/117506183","html_url":"https://github.com/aodn/compliance-checker/issues/12#issuecomment-117506183","issue_url":"https://api.github.com/repos/aodn/compliance-checker/issues/12","id":117506183,"user":{"login":"ggalibert","id":3498686,"avatar_url":"https://avatars.githubusercontent.com/u/3498686?v=3","gravatar_id":"","url":"https://api.github.com/users/ggalibert","html_url":"https://github.com/ggalibert","followers_url":"https://api.github.com/users/ggalibert/followers","following_url":"https://api.github.com/users/ggalibert/following{/other_user}","gists_url":"https://api.github.com/users/ggalibert/gists{/gist_id}","starred_url":"https://api.github.com/users/ggalibert/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ggalibert/subscriptions","organizations_url":"https://api.github.com/users/ggalibert/orgs","repos_url":"https://api.github.com/users/ggalibert/repos","events_url":"https://api.github.com/users/ggalibert/events{/privacy}","received_events_url":"https://api.github.com/users/ggalibert/received_events","type":"User","site_admin":false},"created_at":"2015-07-01T07:33:30Z","updated_at":"2015-07-01T07:33:30Z","body":"Sometimes, an instrument file without pressure information has a temporal extent greater than the two nearest instrument files with pressure information. In this case, NaNs are introduced in the result of the interpolated DEPTH to possibly pad both ends of the dataset without pressure information.\r\n\r\nI'm ok for solution 1).\r\n\r\nIf DEPTH is not listed as an auxiliary coordinate anymore in the coordinates variable attribute then it could have a coordinate variable attribute itself. @mhidas ?"}}