Skip to content
This repository has been archived by the owner on Feb 7, 2023. It is now read-only.

[Python3] Python3 update master task #315

Closed
Yangqing opened this issue Apr 19, 2017 · 8 comments
Closed

[Python3] Python3 update master task #315

Yangqing opened this issue Apr 19, 2017 · 8 comments

Comments

@Yangqing
Copy link
Contributor

Currently, Caffe2 is supporting 2.7 only and python3 support is coming. Setting up this task to have a single point for status checking and updates.

@rachitgarg91
Copy link

rachitgarg91 commented Apr 21, 2017

Hello,

python --version shows 2.7, however, caffe2 uses python from another path:

-- Found PythonInterp: /Users/Innovate/anaconda/bin/python (found version "3.5.2")
...
-- BUILD_PYTHON : ON
-- Python version : 3.5.2
-- Python library : /Users/Innovate/anaconda/lib/libpython3.5m.dylib

which python
/Users/Innovate/anaconda/envs/py3k/bin/python

Where to set the path to python for caffe2?

Thanks!

@eddy-ilg
Copy link

eddy-ilg commented May 5, 2017

One criteria for our switch to Caffe2 or using a non-caffe framework instead is Python3. I don't want to bother, but do you have an estimate of when it might be available? (Weeks, Months, Years ... ;)

@lukeyeager
Copy link
Contributor

Once Python2 testing is enabled with #817, it should be pretty simple to add another target to test python3 with every commit.

@dheera
Copy link

dheera commented Jul 27, 2017

Any update on python3 support? Our entire system is python3 and it would be a pity to not be able to use caffe2 because of this. caffe, tensorflow, and pretty much all other frameworks support python3.

:) python3 -c 'from caffe2.python import core'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/caffe2/python/core.py", line 9, in <module>
    from past.builtins import basestring
ImportError: No module named 'past'

@Yangqing
Copy link
Contributor Author

Yangqing commented Aug 1, 2017

Hey guys, quick update - our internal testing of python3 support seems to suggest that we are python3 compatible, so with a few more tests we will mark this resolved. If you'd like to give it a short, please do and let us know if things work incorrectly.

@dheera - your issue is because you'll need to install future:

https://pypi.python.org/pypi/future

@pietern
Copy link
Contributor

pietern commented Dec 12, 2017

We now run Python 3 builds for every commit. I think it's safe to close this issue now.

@pietern pietern closed this as completed Dec 12, 2017
@dheera
Copy link

dheera commented Dec 12, 2017

Awesome thanks all!

@garvita-tiwari
Copy link

I am still getting this error
Makefile:4: recipe for target 'all' failed
I tried with python 3 and 2 both.
Any suggestions?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants