Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Python 3 support via future package #466

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Add Python 3 support via future package #466

wants to merge 21 commits into from

Conversation

gokceneraslan
Copy link

@gokceneraslan gokceneraslan commented Sep 3, 2016

  • Python3 support added using futurize and some manual tweaking.
  • WebError pyramid dependency is dropped since it's not Python3 compatible. pyramid_debugtoolbar is used by default.
  • Paste is dropped and web server is replaced with waitress which is the default server in pyramid.
  • CMakeLists.txt is updated to detect boost-python3 module if python3 is used to build moe.
  • All tests pass in both Python 2 and 3. Simple tests with easy_interface work nicely.

Some help is needed for updating conda-recipe (if needed), testing Python3 with docker and updating documentation properly. Also proper testing of the web interface is required.

As a future task, splitting client and server implementations into two python packages would be really convenient, IMHO. Clients should not bother with cmake and rest of the server implementation.

@jprobichaud
Copy link

sorry to "necro-bump" but is this finally merged/fixed? It seems I'm having issues when attempting to build/use MOE with python3 as the default python version

@gokceneraslan
Copy link
Author

I guess this is not maintained any more. Maybe someone should help porting https://github.com/wujian16/Cornell-MOE to Python 3.

@jprobichaud
Copy link

jprobichaud commented Nov 7, 2017 via email

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

Successfully merging this pull request may close these issues.

2 participants