{"action":"created","issue":{"url":"https://api.github.com/repos/SHTOOLS/SHTOOLS/issues/17","repository_url":"https://api.github.com/repos/SHTOOLS/SHTOOLS","labels_url":"https://api.github.com/repos/SHTOOLS/SHTOOLS/issues/17/labels{/name}","comments_url":"https://api.github.com/repos/SHTOOLS/SHTOOLS/issues/17/comments","events_url":"https://api.github.com/repos/SHTOOLS/SHTOOLS/issues/17/events","html_url":"https://github.com/SHTOOLS/SHTOOLS/pull/17","id":160788344,"number":17,"title":"Python 3 support","user":{"login":"QuLogic","id":302469,"avatar_url":"https://avatars.githubusercontent.com/u/302469?v=3","gravatar_id":"","url":"https://api.github.com/users/QuLogic","html_url":"https://github.com/QuLogic","followers_url":"https://api.github.com/users/QuLogic/followers","following_url":"https://api.github.com/users/QuLogic/following{/other_user}","gists_url":"https://api.github.com/users/QuLogic/gists{/gist_id}","starred_url":"https://api.github.com/users/QuLogic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/QuLogic/subscriptions","organizations_url":"https://api.github.com/users/QuLogic/orgs","repos_url":"https://api.github.com/users/QuLogic/repos","events_url":"https://api.github.com/users/QuLogic/events{/privacy}","received_events_url":"https://api.github.com/users/QuLogic/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2016-06-17T00:13:26Z","updated_at":"2016-06-21T20:28:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/SHTOOLS/SHTOOLS/pulls/17","html_url":"https://github.com/SHTOOLS/SHTOOLS/pull/17","diff_url":"https://github.com/SHTOOLS/SHTOOLS/pull/17.diff","patch_url":"https://github.com/SHTOOLS/SHTOOLS/pull/17.patch"},"body":"~~This is based on #16, so the Makefile changes are a bit larger than they should be. I can rebased after that is merged.~~\r\n\r\nThis is rather straightforward, really only the `print` function and a few minor other changes. The `python` targets will build for both 2 and 3 by default (but this can be changed by setting the `PYTHON_VERSION` variable). To build just one or the other, there are `python2` an `python3` versions of the targets as well."},"comment":{"url":"https://api.github.com/repos/SHTOOLS/SHTOOLS/issues/comments/227561625","html_url":"https://github.com/SHTOOLS/SHTOOLS/pull/17#issuecomment-227561625","issue_url":"https://api.github.com/repos/SHTOOLS/SHTOOLS/issues/17","id":227561625,"user":{"login":"QuLogic","id":302469,"avatar_url":"https://avatars.githubusercontent.com/u/302469?v=3","gravatar_id":"","url":"https://api.github.com/users/QuLogic","html_url":"https://github.com/QuLogic","followers_url":"https://api.github.com/users/QuLogic/followers","following_url":"https://api.github.com/users/QuLogic/following{/other_user}","gists_url":"https://api.github.com/users/QuLogic/gists{/gist_id}","starred_url":"https://api.github.com/users/QuLogic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/QuLogic/subscriptions","organizations_url":"https://api.github.com/users/QuLogic/orgs","repos_url":"https://api.github.com/users/QuLogic/repos","events_url":"https://api.github.com/users/QuLogic/events{/privacy}","received_events_url":"https://api.github.com/users/QuLogic/received_events","type":"User","site_admin":false},"created_at":"2016-06-21T20:28:50Z","updated_at":"2016-06-21T20:28:50Z","body":"Sure, it's probably better to define `SYSPYPATH` the same way as well.\r\n\r\nThe library extension is correct; it is defined by [PEP 3149](https://www.python.org/dev/peps/pep-3149/) to allow for parallel installation of C extensions (and bytecode) for different runtimes.\r\n\r\n> we can replace the definitions for F2PY and F2PY3 in the makefile with the following:\r\n> ```\r\n> python -m numpy.f2py\r\n> pyhton3 -m numpy.f2py\r\n> ```\r\n\r\nUnfortunately, this is not portable as it depends on which release of NumPy you are using. With 1.9.2, I get:\r\n```\r\npython -m numpy.f2py --quiet -Imodules -Llib --f90flags=\"-m64 -fPIC -O3 -ffast-math\" \\\r\n\t-c src/pyshtools.pyf src/PythonWrapper.f95\\\r\n\t-lSHTOOLS -L/usr/local/lib -lfftw3 -lm -llapack  -lblas\r\n/usr/bin/python: No module named numpy.f2py.__main__; 'numpy.f2py' is a package and cannot be directly executed\r\nMakefile:280: recipe for target 'pyshtools/_SHTOOLS.so' failed\r\nmake: *** [pyshtools/_SHTOOLS.so] Error 1\r\n```\r\n\r\n> It is not clear to me if these are temporary files that should be deleted, or if there is a reason to keep them....\r\n\r\nI don't really know. The extension suggests it's macOS-specific. I guess you can see if any other `.so` have a corresponding `.so.dSYM` directory in your existing Python installation?"}}
{"actor":{"display_login":"QuLogic"}}