python module

[vchahun]

• not using binary LM anymore
• I converted README to markdown, keep it if you like...

[kpu]

Stupid question: should that auto-generated kenlm.cpp be part of the source code?

[vchahun]

Yes, this is the recommendation from cython:
http://docs.cython.org/src/userguide/source_files_and_compilation.html#distributing-cython-modules

This allows installation without cython. When the cython source is modified, you just need to run:

cython --cplus python/kenlm.pyx

[hieuhoang]

Hi victor,

I'm revelling at the moment so not able to look at your fork. What's
the difference between yours and ken's version?

If u have some useful contribution you're better off telling ken to
merge it with his version. I don't control that part of Moses

Hieu
Sent from my flying horse

On 4 Jul 2012, at 02:43 AM, Victor Chahuneau
reply@reply.github.com
wrote:

• not using binary LM anymore
• I converted README to markdown, keep it if you like...

You can merge this Pull Request by running:

git pull https://github.com/vchahun/kenlm master

Or you can view, comment on it, or merge it online at:

#2

-- Commit Summary --

• Add python module; supports LM querying
• Some instructions
• Fix for g++ and python 3
• [python] Change to non-binary LM in example; catch C++ exception
• Merge READMEs

-- File Changes --

R README.md (44)
A examples/example.py (26)
A python/kenlm.cpp (3436)
A python/kenlm.pxd (28)
A python/kenlm.pyx (65)
A setup.py (24)

-- Patch Links --

https://github.com/kpu/kenlm/pull/2.patch
https://github.com/kpu/kenlm/pull/2.diff

---

Reply to this email directly or view it on GitHub:
#2

[kpu]

Hi Hieu,

This was a pull request for kenlm that I asked Victor to make.  You 

have commit access to kenlm, so github spammed you. It's not the
mosesdecoder project.

Kenneth

On 07/04/2012 08:47 AM, Hieu Hoang wrote:

Hi victor,

I'm revelling at the moment so not able to look at your fork. What's
the difference between yours and ken's version?

If u have some useful contribution you're better off telling ken to
merge it with his version. I don't control that part of Moses

Hieu
Sent from my flying horse

On 4 Jul 2012, at 02:43 AM, Victor Chahuneau
reply@reply.github.com
wrote:

• not using binary LM anymore
• I converted README to markdown, keep it if you like...

You can merge this Pull Request by running:

git pull https://github.com/vchahun/kenlm master

Or you can view, comment on it, or merge it online at:

#2

-- Commit Summary --

• Add python module; supports LM querying
• Some instructions
• Fix for g++ and python 3
• [python] Change to non-binary LM in example; catch C++ exception
• Merge READMEs

-- File Changes --

R README.md (44)
A examples/example.py (26)
A python/kenlm.cpp (3436)
A python/kenlm.pxd (28)
A python/kenlm.pyx (65)
A setup.py (24)

-- Patch Links --

https://github.com/kpu/kenlm/pull/2.patch
https://github.com/kpu/kenlm/pull/2.diff

---

Reply to this email directly or view it on GitHub:
#2

---

Reply to this email directly or view it on GitHub:
#2 (comment)

[hieuhoang]

oh, damn this conniving github & all it's offsprings

On 4 July 2012 14:12, Kenneth Heafield <
reply@reply.github.com

wrote:

Hi Hieu,

    This was a pull request for kenlm that I asked Victor to make.  You

have commit access to kenlm, so github spammed you. It's not the
mosesdecoder project.

Kenneth

On 07/04/2012 08:47 AM, Hieu Hoang wrote:

Hi victor,

I'm revelling at the moment so not able to look at your fork. What's
the difference between yours and ken's version?

If u have some useful contribution you're better off telling ken to
merge it with his version. I don't control that part of Moses

Hieu
Sent from my flying horse

On 4 Jul 2012, at 02:43 AM, Victor Chahuneau
reply@reply.github.com
wrote:

• not using binary LM anymore
• I converted README to markdown, keep it if you like...

You can merge this Pull Request by running:

git pull https://github.com/vchahun/kenlm master

Or you can view, comment on it, or merge it online at:

#2

-- Commit Summary --

• Add python module; supports LM querying
• Some instructions
• Fix for g++ and python 3
• [python] Change to non-binary LM in example; catch C++ exception
• Merge READMEs

-- File Changes --

R README.md (44)
A examples/example.py (26)
A python/kenlm.cpp (3436)
A python/kenlm.pxd (28)
A python/kenlm.pyx (65)
A setup.py (24)

-- Patch Links --

https://github.com/kpu/kenlm/pull/2.patch
https://github.com/kpu/kenlm/pull/2.diff

---

Reply to this email directly or view it on GitHub:
#2

---

Reply to this email directly or view it on GitHub:
#2 (comment)

---

Reply to this email directly or view it on GitHub:
#2 (comment)