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

Any plan to upgrade to python 3.x? #10

Open
mfkilgore opened this issue Jan 18, 2016 · 5 comments
Open

Any plan to upgrade to python 3.x? #10

mfkilgore opened this issue Jan 18, 2016 · 5 comments

Comments

@mfkilgore
Copy link

I have made a few changes myself as required, but would be nice to have 3.x supported too.

Thanks,

@JnBrymn-EB
Copy link
Contributor

I agree it would be nice to have support for 3.x, though it's difficult because I don't want the distinction between 2.x and 3.x to be a cause of confusion itself. Do you see any place where the code is currently 2.x compatible and not 3.x compatible but can easily be made compatible with both?

@softwaredoug
Copy link
Collaborator

I'm guessing this would mostly be print statements...

On Tuesday, January 19, 2016, John Berryman notifications@github.com
wrote:

I agree it would be nice to have support for 3.x, though it's difficult
because I don't want the distinction between 2.x and 3.x to be a cause of
confusion itself. Do you see any place where the code is currently 2.x
compatible and not 3.x compatible but can easily be made compatible with
both?


Reply to this email directly or view it on GitHub
#10 (comment)
.

@mfkilgore
Copy link
Author

True most of the changes are print statements and simple to fix. There are a couple of others like iteritems -> items, but mostly easy.

@villasv
Copy link

villasv commented Oct 30, 2018

I've been porting to Python 3 and Elasticsearch 6 in my own fork but I'm taking full liberties to rename and remove stuff so I don't think I'll ever actually make a PR.

But yes, the effort is very small. Mostly print statements and iteritems for python, and mostly get parameters that became json parameters or string -> text types on the ES side. Besides the classic similarity metric being completely deprecated in 6.3.0.

@softwaredoug
Copy link
Collaborator

softwaredoug commented Oct 30, 2018 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

No branches or pull requests

4 participants