Crawford Comeaux

@rchari51 This is what I get after manually making the changes to `spacy/tokens/doc.pyx` and `lda2vec/preprocess.py`: ``` Traceback (most recent call last): File "data/preprocess.py", line 47, in merge=True) File "/home/ubuntu/lda2vec/lda2vec/preprocess.py", line...

@saravp Only with `merge=False`, which doesn't really fit my use case.

@saravp I just took a look at spaCy's issues to see if anything related to this stuck out & they just shipped version 1.0. Does anything change if you update...

@chenjun0210 Your issue might be fixed via #46. Apply the changes to corpus.py & also confirm you have the packages mentioned in the PR's first comment.

I've run into similar issues (or the same issue) where merge=False resolves things, but what impact does that have on the results besides squashing the error?

I say go for it!

Is it premature to start pushing for porting with package management systems? Disclaimer: I'm not familiar with the process of packaging libraries for things like pip, rubygems, npm, etc.— Sent...

+1 — Sent from Mailbox for iPhone, suckah! On Mon, Jul 22, 2013 at 10:31 PM, Oleg Seletsky [email protected] wrote: > @akalicki , @jehna , > Perhaps it would be...

I started working on this, too. I'll take a look at your code in a second, but I recommend digging into the source for XRegExp (https://github.com/slevithan/xregexp). It augments js regexes...

I think the way xregexp handles named captures makes sense, specifically: - bare integers aren't allowed, though they can be used as string names - named captures can be back-referenced...