{"action":"created","issue":{"url":"https://api.github.com/repos/spacetelescope/poppy/issues/170","repository_url":"https://api.github.com/repos/spacetelescope/poppy","labels_url":"https://api.github.com/repos/spacetelescope/poppy/issues/170/labels{/name}","comments_url":"https://api.github.com/repos/spacetelescope/poppy/issues/170/comments","events_url":"https://api.github.com/repos/spacetelescope/poppy/issues/170/events","html_url":"https://github.com/spacetelescope/poppy/issues/170","id":353584303,"node_id":"MDU6SXNzdWUzNTM1ODQzMDM=","number":170,"title":"Issues with astropy-helpers","user":{"login":"mperrin","id":1151745,"node_id":"MDQ6VXNlcjExNTE3NDU=","avatar_url":"https://avatars0.githubusercontent.com/u/1151745?v=4","gravatar_id":"","url":"https://api.github.com/users/mperrin","html_url":"https://github.com/mperrin","followers_url":"https://api.github.com/users/mperrin/followers","following_url":"https://api.github.com/users/mperrin/following{/other_user}","gists_url":"https://api.github.com/users/mperrin/gists{/gist_id}","starred_url":"https://api.github.com/users/mperrin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mperrin/subscriptions","organizations_url":"https://api.github.com/users/mperrin/orgs","repos_url":"https://api.github.com/users/mperrin/repos","events_url":"https://api.github.com/users/mperrin/events{/privacy}","received_events_url":"https://api.github.com/users/mperrin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-08-23T22:59:49Z","updated_at":"2018-08-23T22:59:52Z","closed_at":null,"author_association":"COLLABORATOR","body":"<a href=\"https://github.com/mperrin\"><img src=\"https://avatars0.githubusercontent.com/u/1151745?v=4\" align=\"left\" width=\"96\" height=\"96\" hspace=\"10\"></img></a> **Issue by [mperrin](https://github.com/mperrin)**\n_Friday Jun 10, 2016 at 16:45 GMT_\n_Originally opened as https://github.com/mperrin/poppy/issues/170_\n\n----\n\nsplitting off this topic from https://github.com/mperrin/poppy/issues/160#issuecomment-225229716\n\nHi @eteq, @josePhoenix can give more of the details, but I think it's a bit of both. \n\nFor literally years, every so often I've encountered some random and hard-to-debug failures which eventually traced back to \"the astropy helpers are doing something unexpected behind the scenes\":\n- https://github.com/astropy/package-template/issues/106\n- https://github.com/astropy/astropy/issues/2502\n- https://github.com/astropy/astropy/pull/3713\n- and now here this new \"the astropy helpers are using version 2.7 of py.test even though I've installed 2.8\"\n- which is possibly related to why we're getting consistent failures on Travis on python 3.5, even though all tests pass on 3.5 locally. https://github.com/mperrin/poppy/issues/159\n\nEach of the above has led to time-consuming debugging to track down what's going on, which has been a time sink and distraction from the main tasks we're actually trying to get done. I readily admit that software is hard, and we can't expect a bug-free existence or anything close to it. But overall given the limited functionality we're actually gaining from the astropy helpers (mostly some setup.py tweaks and the sphinx extensions), it seems like the overall return on time invested is negative.  \n\n"},"comment":{"url":"https://api.github.com/repos/spacetelescope/poppy/issues/comments/415598576","html_url":"https://github.com/spacetelescope/poppy/issues/170#issuecomment-415598576","issue_url":"https://api.github.com/repos/spacetelescope/poppy/issues/170","id":415598576,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNTU5ODU3Ng==","user":{"login":"mperrin","id":1151745,"node_id":"MDQ6VXNlcjExNTE3NDU=","avatar_url":"https://avatars0.githubusercontent.com/u/1151745?v=4","gravatar_id":"","url":"https://api.github.com/users/mperrin","html_url":"https://github.com/mperrin","followers_url":"https://api.github.com/users/mperrin/followers","following_url":"https://api.github.com/users/mperrin/following{/other_user}","gists_url":"https://api.github.com/users/mperrin/gists{/gist_id}","starred_url":"https://api.github.com/users/mperrin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mperrin/subscriptions","organizations_url":"https://api.github.com/users/mperrin/orgs","repos_url":"https://api.github.com/users/mperrin/repos","events_url":"https://api.github.com/users/mperrin/events{/privacy}","received_events_url":"https://api.github.com/users/mperrin/received_events","type":"User","site_admin":false},"created_at":"2018-08-23T22:59:52Z","updated_at":"2018-08-23T22:59:52Z","author_association":"COLLABORATOR","body":"<a href=\"https://github.com/josePhoenix\"><img src=\"https://avatars3.githubusercontent.com/u/172886?v=4\" align=\"left\" width=\"48\" height=\"48\" hspace=\"10\"></img></a> **Comment by [josePhoenix](https://github.com/josePhoenix)**\n_Friday Aug 11, 2017 at 18:04 GMT_\n\n----\n\nAstropy helpers and the package template strike again!\r\n\r\nWhen building WebbPSF on ReadTheDocs, our latest version of POPPY (**including** the latest v2.0.1 astropy_helpers submodule) was unable to install:\r\n\r\n\r\nExcerpt from https://readthedocs.org/projects/webbpsf/builds/5821195/:\r\n\r\n```\r\n[ snip ]\r\nSearching for poppy>=0.5.0\r\nReading https://pypi.python.org/simple/poppy/\r\nDownloading https://pypi.python.org/packages/22/39/6ced910b3355a7f7e764cb985b24dac8b9a46c215fb9c1461145896c57d5/poppy-0.6.0.tar.gz#md5=249065f2f7892077acb321d1bc646002\r\nBest match: poppy 0.6.0\r\nProcessing poppy-0.6.0.tar.gz\r\nWriting /tmp/easy_install-ha9cewkv/poppy-0.6.0/setup.cfg\r\nRunning poppy-0.6.0/setup.py -q bdist_egg --dist-dir /tmp/easy_install-ha9cewkv/poppy-0.6.0/egg-dist-tmp-nwihqd2i\r\n/home/docs/checkouts/readthedocs.org/user_builds/webbpsf/envs/stable/lib/python3.5/site-packages/pkg_resources/__init__.py:188: RuntimeWarning: You have iterated over the result of pkg_resources.parse_version. This is a legacy behavior which is inconsistent with the new version class introduced in setuptools 8.0. In most cases, conversion to a tuple is unnecessary. For comparison of versions, sort the Version instances directly. If you have another use case requiring the tuple, please file a bug with the setuptools project describing that need.\r\n  stacklevel=1,\r\nTraceback (most recent call last):\r\n  File \"/home/docs/checkouts/readthedocs.org/user_builds/webbpsf/envs/stable/lib/python3.5/site-packages/setuptools/sandbox.py\", line 157, in save_modules\r\n    yield saved\r\n  File \"/home/docs/checkouts/readthedocs.org/user_builds/webbpsf/envs/stable/lib/python3.5/site-packages/setuptools/sandbox.py\", line 198, in setup_context\r\n    yield\r\n  File \"/home/docs/checkouts/readthedocs.org/user_builds/webbpsf/envs/stable/lib/python3.5/site-packages/setuptools/sandbox.py\", line 248, in run_setup\r\n    DirectorySandbox(setup_dir).run(runner)\r\n  File \"/home/docs/checkouts/readthedocs.org/user_builds/webbpsf/envs/stable/lib/python3.5/site-packages/setuptools/sandbox.py\", line 278, in run\r\n    return func()\r\n  File \"/home/docs/checkouts/readthedocs.org/user_builds/webbpsf/envs/stable/lib/python3.5/site-packages/setuptools/sandbox.py\", line 246, in runner\r\n    _execfile(setup_script, ns)\r\n  File \"/home/docs/checkouts/readthedocs.org/user_builds/webbpsf/envs/stable/lib/python3.5/site-packages/setuptools/sandbox.py\", line 47, in _execfile\r\n    exec(code, globals, locals)\r\n  File \"/tmp/easy_install-ha9cewkv/poppy-0.6.0/temp/easy_install-kmr20fds/astropy-helpers-0.4.8/setup.py\", line 49, in <module>\r\n    AUTHOR_EMAIL = metadata.get('author_email', '')\r\n  File \"/home/docs/.pyenv/versions/3.5.2/lib/python3.5/distutils/core.py\", line 134, in setup\r\n    ok = dist.parse_command_line()\r\n  File \"/home/docs/checkouts/readthedocs.org/user_builds/webbpsf/envs/stable/lib/python3.5/site-packages/setuptools/dist.py\", line 347, in parse_command_line\r\n    result = _Distribution.parse_command_line(self)\r\n  File \"/home/docs/.pyenv/versions/3.5.2/lib/python3.5/distutils/dist.py\", line 472, in parse_command_line\r\n    args = self._parse_command_opts(parser, args)\r\n  File \"/home/docs/checkouts/readthedocs.org/user_builds/webbpsf/envs/stable/lib/python3.5/site-packages/setuptools/dist.py\", line 658, in _parse_command_opts\r\n    nargs = _Distribution._parse_command_opts(self, parser, args)\r\n  File \"/home/docs/.pyenv/versions/3.5.2/lib/python3.5/distutils/dist.py\", line 536, in _parse_command_opts\r\n    \"command class %s must subclass Command\" % cmd_class)\r\ndistutils.errors.DistutilsClassError: command class <class 'setuptools.command.bdist_egg.bdist_egg'> must subclass Command\r\n```\r\n\r\nI saw something on the package template or helpers issue tracker that made me think `ah_bootstrap.py` is out of date. I went to astropy/package-template to find it has been reimagined as a Cookiecutter template. The file `ah_bootstrap.py` is no longer present.\r\n\r\nUpon investigating the post-create hook in the Cookiecutter template, I see it's now in the helpers and only copied out on project creation.\r\n\r\nSo, I updated it in POPPY and ran the RTD build against POPPY `master` and that seemed to fix it.\n"}}
{"actor":{"display_login":"mperrin"}}