{"action":"created","issue":{"url":"https://api.github.com/repos/PCMDI/cmor/issues/385","repository_url":"https://api.github.com/repos/PCMDI/cmor","labels_url":"https://api.github.com/repos/PCMDI/cmor/issues/385/labels{/name}","comments_url":"https://api.github.com/repos/PCMDI/cmor/issues/385/comments","events_url":"https://api.github.com/repos/PCMDI/cmor/issues/385/events","html_url":"https://github.com/PCMDI/cmor/issues/385","id":346994670,"node_id":"MDU6SXNzdWUzNDY5OTQ2NzA=","number":385,"title":"Python error handling no longer works on master","user":{"login":"ehogan","id":918142,"node_id":"MDQ6VXNlcjkxODE0Mg==","avatar_url":"https://avatars1.githubusercontent.com/u/918142?v=4","gravatar_id":"","url":"https://api.github.com/users/ehogan","html_url":"https://github.com/ehogan","followers_url":"https://api.github.com/users/ehogan/followers","following_url":"https://api.github.com/users/ehogan/following{/other_user}","gists_url":"https://api.github.com/users/ehogan/gists{/gist_id}","starred_url":"https://api.github.com/users/ehogan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ehogan/subscriptions","organizations_url":"https://api.github.com/users/ehogan/orgs","repos_url":"https://api.github.com/users/ehogan/repos","events_url":"https://api.github.com/users/ehogan/events{/privacy}","received_events_url":"https://api.github.com/users/ehogan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-08-02T12:42:36Z","updated_at":"2018-08-03T19:37:54Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"Hi @doutriaux1, @taylor13,\r\n\r\nI was attempting to follow up with #245 by running some tests with master and I realised that the Python error handling I introduced in #86 no longer works; now when I run CMOR, instead of seeing, e.g.\r\n\r\n```\r\nTraceback (most recent call last):\r\n  File \"./example.py\", line 43, in main\r\n    cmor.write(var_id, data[i:i])\r\n  File \"/cmor/lib/python2.7/site-packages/CMOR-3.3-py2.7-linux-x86_64.egg/cmor/pywrapper.py\", line 800, in write\r\n    time_vals, time_bnds, store_with)\r\n_cmor.CMORError: Problem with 'cmor.write'. Please check the logfile (if defined).\r\n```\r\n\r\n(from CMOR 3.3.0), I now see:\r\n\r\n```\r\nTraceback (most recent call last):\r\n  File \"./example.py\", line 43, in main\r\n    cmor.write(var_id, data[i:i])\r\n  File \"/cmor/lib/python2.7/site-packages/CMOR-3.3.3-py2.7-linux-x86_64.egg/cmor/pywrapper.py\", line 817, in write\r\n    time_vals, time_bnds, store_with)\r\nKeyboardInterrupt\r\n```\r\n\r\nwhich was exactly the issue I was trying to avoid!\r\n\r\nIn #374 @doutriaux1 [mentioned](https://github.com/PCMDI/cmor/issues/374#issuecomment-408886577) that a change was necessary for Fortran to exit nicely. Would it be possible to fix the Fortran issue without affecting the way the Python error handling works, please? :)"},"comment":{"url":"https://api.github.com/repos/PCMDI/cmor/issues/comments/410355474","html_url":"https://github.com/PCMDI/cmor/issues/385#issuecomment-410355474","issue_url":"https://api.github.com/repos/PCMDI/cmor/issues/385","id":410355474,"node_id":"MDEyOklzc3VlQ29tbWVudDQxMDM1NTQ3NA==","user":{"login":"taylor13","id":4993439,"node_id":"MDQ6VXNlcjQ5OTM0Mzk=","avatar_url":"https://avatars3.githubusercontent.com/u/4993439?v=4","gravatar_id":"","url":"https://api.github.com/users/taylor13","html_url":"https://github.com/taylor13","followers_url":"https://api.github.com/users/taylor13/followers","following_url":"https://api.github.com/users/taylor13/following{/other_user}","gists_url":"https://api.github.com/users/taylor13/gists{/gist_id}","starred_url":"https://api.github.com/users/taylor13/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/taylor13/subscriptions","organizations_url":"https://api.github.com/users/taylor13/orgs","repos_url":"https://api.github.com/users/taylor13/repos","events_url":"https://api.github.com/users/taylor13/events{/privacy}","received_events_url":"https://api.github.com/users/taylor13/received_events","type":"User","site_admin":false},"created_at":"2018-08-03T19:37:54Z","updated_at":"2018-08-03T19:37:54Z","author_association":"NONE","body":"If @matthew-mizielinski and @ehogan are happy, it's fine with me. Perhaps Emma could test it before we do the next release. "}}
{"actor":{"display_login":"taylor13"}}