{"action":"created","issue":{"url":"https://api.github.com/repos/ghrcdaac/cmr/issues/17","repository_url":"https://api.github.com/repos/ghrcdaac/cmr","labels_url":"https://api.github.com/repos/ghrcdaac/cmr/issues/17/labels{/name}","comments_url":"https://api.github.com/repos/ghrcdaac/cmr/issues/17/comments","events_url":"https://api.github.com/repos/ghrcdaac/cmr/issues/17/events","html_url":"https://github.com/ghrcdaac/cmr/pull/17","id":221412275,"number":17,"title":"pyCMR Overhaul","user":{"login":"lewismc","id":1165719,"avatar_url":"https://avatars2.githubusercontent.com/u/1165719?v=3","gravatar_id":"","url":"https://api.github.com/users/lewismc","html_url":"https://github.com/lewismc","followers_url":"https://api.github.com/users/lewismc/followers","following_url":"https://api.github.com/users/lewismc/following{/other_user}","gists_url":"https://api.github.com/users/lewismc/gists{/gist_id}","starred_url":"https://api.github.com/users/lewismc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lewismc/subscriptions","organizations_url":"https://api.github.com/users/lewismc/orgs","repos_url":"https://api.github.com/users/lewismc/repos","events_url":"https://api.github.com/users/lewismc/events{/privacy}","received_events_url":"https://api.github.com/users/lewismc/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2017-04-12T22:50:58Z","updated_at":"2017-04-14T16:32:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ghrcdaac/cmr/pulls/17","html_url":"https://github.com/ghrcdaac/cmr/pull/17","diff_url":"https://github.com/ghrcdaac/cmr/pull/17.diff","patch_url":"https://github.com/ghrcdaac/cmr/pull/17.patch"},"body":"Hi Folks,\r\nThis PR essentially overhauls pyCMR setup.py and codebase to build and run on Python 3.X. Additionally, it relocates the **test** directory to a new location hence cleaning up the directory structure.\r\nTests are currently failing due to #16 \r\n```\r\n$ nosetests\r\n...\r\n======================================================================\r\nERROR: test suite for <class 'pyCMR.tests.test_cmr_integration.TestCMRIntegration'>\r\n----------------------------------------------------------------------\r\nTraceback (most recent call last):\r\n  File \"/Users/lmcgibbn/miniconda3/lib/python3.5/site-packages/nose/suite.py\", line 210, in run\r\n    self.setUp()\r\n  File \"/Users/lmcgibbn/miniconda3/lib/python3.5/site-packages/nose/suite.py\", line 293, in setUp\r\n    self.setupContext(ancestor)\r\n  File \"/Users/lmcgibbn/miniconda3/lib/python3.5/site-packages/nose/suite.py\", line 316, in setupContext\r\n    try_run(context, names)\r\n  File \"/Users/lmcgibbn/miniconda3/lib/python3.5/site-packages/nose/util.py\", line 471, in try_run\r\n    return func()\r\n  File \"/usr/local/cmr/pyCMR/tests/test_cmr_integration.py\", line 18, in setUpClass\r\n    cls.cmr = CMR('cmr.cfg')\r\n  File \"/usr/local/cmr/pyCMR/pyCMR.py\", line 65, in __init__\r\n    raise IOError(\"The config file can't be opened for reading/writing\")\r\nOSError: The config file can't be opened for reading/writing\r\n\r\n----------------------------------------------------------------------\r\nRan 0 tests in 0.043s\r\n\r\nFAILED (errors=1)\r\n```"},"comment":{"url":"https://api.github.com/repos/ghrcdaac/cmr/issues/comments/294186229","html_url":"https://github.com/ghrcdaac/cmr/pull/17#issuecomment-294186229","issue_url":"https://api.github.com/repos/ghrcdaac/cmr/issues/17","id":294186229,"user":{"login":"mileswwatkins","id":4959135,"avatar_url":"https://avatars2.githubusercontent.com/u/4959135?v=3","gravatar_id":"","url":"https://api.github.com/users/mileswwatkins","html_url":"https://github.com/mileswwatkins","followers_url":"https://api.github.com/users/mileswwatkins/followers","following_url":"https://api.github.com/users/mileswwatkins/following{/other_user}","gists_url":"https://api.github.com/users/mileswwatkins/gists{/gist_id}","starred_url":"https://api.github.com/users/mileswwatkins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mileswwatkins/subscriptions","organizations_url":"https://api.github.com/users/mileswwatkins/orgs","repos_url":"https://api.github.com/users/mileswwatkins/repos","events_url":"https://api.github.com/users/mileswwatkins/events{/privacy}","received_events_url":"https://api.github.com/users/mileswwatkins/received_events","type":"User","site_admin":false},"created_at":"2017-04-14T16:32:25Z","updated_at":"2017-04-14T16:32:25Z","body":"> IMHO we should merge these efforts and transition the unified toolkit over to the NASA Github Org\r\n\r\nI'd agree that that's the ideal plan, and that combining the code wouldn't be too bad,  but I think the catch is that _both libraries_ already have uses in production, and so altering the API of either, much less merging them and moving to a new address, has a nontrivial political cost. If you can swing it, power to ya ;D"}}
{"actor":{"display_login":"mileswwatkins"}}