{"action":"closed","issue":{"url":"https://api.github.com/repos/PCMDI/cmor/issues/214","repository_url":"https://api.github.com/repos/PCMDI/cmor","labels_url":"https://api.github.com/repos/PCMDI/cmor/issues/214/labels{/name}","comments_url":"https://api.github.com/repos/PCMDI/cmor/issues/214/comments","events_url":"https://api.github.com/repos/PCMDI/cmor/issues/214/events","html_url":"https://github.com/PCMDI/cmor/issues/214","id":245045569,"number":214,"title":"CMOR 3.2.5 not compatible with version 01.00.12 of the MIP tables","user":{"login":"ehogan","id":918142,"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":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-07-24T11:06:17Z","updated_at":"2017-07-31T17:23:46Z","closed_at":"2017-07-31T17:23:46Z","body":"The code in [Test/jamie_hybrid_height.py](https://github.com/PCMDI/cmor/blob/master/Test/jamie_hybrid_height.py) completes successfully using CMOR 3.2.4 and version `01.00.12` of the MIP tables. However, CMOR 3.2.5 and version `01.00.12` of the MIP tables raises an exception:\r\n\r\n```\r\n% ./example.py \r\ncmor.axis calls complete\r\nTraceback (most recent call last):\r\n  File \"./example.py\", line 54, in <module>\r\n    main1()\r\n  File \"./example.py\", line 40, in main1\r\n    'd', [0., 0.5], [[0., 0.25], [0.25, 1.]])\r\n  File \"/home/h03/support/software/cmor-3.2.5/lib/python2.7/site-packages/CMOR-3.2-py2.7-linux-x86_64.egg/cmor/pywrapper.py\", line 644, in zfactor\r\n    ndims, axis_ids, type, zfactor_values, zfactor_bounds)\r\n_cmor.CMORError: Problem with 'cmor.zfactor'. Please check the logfile (if defined).\r\n```\r\n\r\nThe log file contains:\r\n\r\n```\r\nC Traceback:\r\n! In function: cmor_variable\r\n! called from: cmor_zfactor\r\n! \r\n\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n!\r\n! Error: Could not find a matching variable for name: 'b'\r\n!\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n\r\n\r\nC Traceback:\r\n! In function: cmor_variable\r\n! called from: cmor_zfactor\r\n! \r\n\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n!\r\n! Error: You defined variable '' (table 6hrLev) with axis id 'hybrid_height' which is not part of this variable acco\r\nrding to your table, it says: ( )\r\n!\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n\r\n\r\nC Traceback:\r\n! In function: cmor_variable\r\n! \r\n\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n!\r\n! Error: Could not find a matching variable for name: 'b_bnds'\r\n!\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n\r\n\r\nC Traceback:\r\n! In function: cmor_variable\r\n! \r\n\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n!\r\n! Error: You defined variable '' (table 6hrLev) with axis id 'hybrid_height' which is not part of this variable according to your table, it says: ( )\r\n!\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n\r\n```\r\nCMOR 3.2.5 and version `01.00.13` of the MIP tables also raises an exception:\r\n\r\n```\r\n% ./example.py \r\ncmor.axis calls complete\r\ncmor.zfactor calls complete\r\ncmor.variable call complete\r\nTraceback (most recent call last):\r\n  File \"./example.py\", line 54, in <module>\r\n    main1()\r\n  File \"./example.py\", line 47, in main1\r\n    cmor.write(varid, values, time_vals=[6.0], time_bnds=[3., 12.])\r\n  File \"/home/h03/support/software/cmor-3.2.5/lib/python2.7/site-packages/CMOR-3.2-py2.7-linux-x86_64.egg/cmor/pywrapper.py\", line 795, 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\nThe log file contains:\r\n\r\n```\r\nC Traceback:\r\n! In function: _CV_ValidateAttribute\r\n! called from: _CV_checkGblAttributes\r\n! called from: cmor_write\r\n! \r\n\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n!\r\n! Error: The attribute \"further_info_url\" could not be validated. \r\n! The current input value is \"http://furtherinfo.es-doc.org/CMIP6.PCMDI.PCMDI-test-1-0.piControl-withism.none.r11i1p\r\n1f1\" which is not valid \r\n! Valid values must match the regular expression:\r\n! \t[\"^http://furtherinfo.es-doc.org/[[:alpha:]]\\{1,\\}$\"  ...] \r\n! \r\n! Check your Control Vocabulary file \"/project/cdds/etc/mip_tables/CMIP6/01.00.13//CMIP6_CV.json\".\r\n! \r\n!\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n\r\n\r\nC Traceback:\r\n! In function: _CV_checkGblAttributes\r\n! called from: cmor_write\r\n! \r\n\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n!\r\n! Error: Please fix required attributes mentioned in\r\n! the warnings/error above and rerun. (aborting!)\r\n! \r\n!\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n\r\n\r\nC Traceback:\r\n! In function: \r\n\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n!\r\n! Error: Cannot continue until you fix the errors listed above: -1\r\n!\r\n!!!!!!!!!!!!!!!!!!!!!!!!!\r\n```\r\n\r\nThere are two issues here:\r\n\r\n1) It is not clear from the [3.2.5 release notes](https://github.com/PCMDI/cmor/releases/tag/CMOR-3.2.5) that compatibility with MIP tables older than `01.00.13` is broken (see, e.g. the message provided in the [3.2.4 release notes](https://github.com/PCMDI/cmor/releases/tag/CMOR-3.2.4)).\r\n\r\n2) CMOR 3.2.5 appears to construct the `further_info_url` internally (it is not defined in the [common_user_input.json](https://github.com/PCMDI/cmor/blob/master/Test/common_user_input.json)), but the regular expression doesn't allow `.` (`[:alpha:]` is equivalent to `[a-zA-Z]`).\r\n\r\nWould it be possible to resolve these issues, please?\r\n\r\n(cc @jonseddon)"}}
{"actor":{"display_login":"dnadeau4"}}