Support Python 3 and run advisor with PyCharm about advisor HOT 7 CLOSED

python3.5 is ok, just with some import error
such as:
from random_search import RandomSearchAlgorithm
to
from .random_search import RandomSearchAlgorithm
will be ok

from advisor.

Thanks @codlife for your reporting. This should be changed to run in both Python 2 and Python 3.

from advisor.

Any status update for Python 3.7 support? Thanks.

from advisor.

I don't test it yet. It's contribution welcome and free feel to send PR.

I may test and try to import as Python 3 for django soon.

from advisor.

Tested on Python 3.6.6 and I got:

macbook:~ andrewssobral$ advisor_admin server start
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.6/bin/advisor_admin", line 7, in <module>
    from advisor_client.commandline.admin_command import main
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/advisor_client/commandline/admin_command.py", line 110
    except subprocess.CalledProcessError, e:
                                        ^
SyntaxError: invalid syntax
macbook:~ andrewssobral$ 

Any status update for this? Thanks

from advisor.

Tested on Python 3.6.6 and I got:

macbook:~ andrewssobral$ advisor_admin server start
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.6/bin/advisor_admin", line 7, in <module>
    from advisor_client.commandline.admin_command import main
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/advisor_client/commandline/admin_command.py", line 110
    except subprocess.CalledProcessError, e:
                                        ^
SyntaxError: invalid syntax
macbook:~ andrewssobral$ 

Any status update for this? Thanks

Change the comma to as. Then it's OK

from advisor.

Thanks @andresmore and @mygao8 . I will update this soon and need more test in Python 2.7 and 3.6 in the future.

Already fix in #21 .

from advisor.