Add support for python3 about langid.py

[saffsd]

Hi Frederik,

Thanks for reporting the issue. There is a branch that works in Python 3 here. I'd very much like to have a single codebase that runs in both Python 2/3 but I've not had a chance to look into how to do this. If you have any experience in this respect any patches would be appreciated.

Cheers,
Marco

[frederik-elwert]

Thanks, works perfectly!

Re 2/3: I guess using six would be the easiest way to achieve that.

[adampl]

That branch is not fully Python 3 compatible.

There is a lot of syntax errors caused by prints without parentheses. To see them, just run:
pip install https://github.com/saffsd/langid.py/archive/python3.zip.

[saffsd]

Hi Adam,

Thanks for getting in touch. You're right that the port is incomplete,
currently only the core classifier langid.py itself has been ported, the
supporting tools for model building have not. You should be able to use the
file langid.py from that branch in a python3 project.

On Thu, Feb 19, 2015 at 10:27 PM, Adam [email protected] wrote:

That branch is not fully Python 3 compatible.

There is a lot of syntax errors caused by prints without parentheses. To
see them, just run:
pip install https://github.com/saffsd/langid.py/archive/python3.zip.

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

[rrshaban]

@adampl Given that Python2 supports print with parantheses, why not just convert all the print statements to parantheses? Or maybe this has already been done, it's been a couple of months.

[adampl]

@rrshaban There may be other incompatibilities as well, prints are just the most obvious ones.

[saffsd]

As of 600f530 the core script langid/langid.py runs in both Python2 and Python3 interpreters. Porting the training tools is a bigger job and not in my workplan at the moment. I'll mark this as closed for now.