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

compatibility with spaCy 3.0 ? #295

Open
d5555 opened this issue Feb 4, 2021 · 32 comments
Open

compatibility with spaCy 3.0 ? #295

d5555 opened this issue Feb 4, 2021 · 32 comments

Comments

@d5555
Copy link

d5555 commented Feb 4, 2021

Is there going to be neuralcoref compatible with spaCy 3.0 ?

@svlandeg
Copy link
Collaborator

svlandeg commented Feb 4, 2021

Yes! This is definitely on our road map :-) I'll actually pin this issue because I'm sure others are interested as well.

@asterbini
Copy link

Plus 1

@asterbini
Copy link

After some rough paches I am able to start neuralcoref with spacy 3 but I obviously get the error
''Trying to read a Model that was created with an incompatible version of Thinc''
Is there a way to convert the models to the new Thinc version used in Spacy 3?

@svlandeg
Copy link
Collaborator

@asterbini : Due to the many internal changes between Thinc 7 and 8, and spaCy 2 and 3, I'm not sure this is going to be feasible. See also this discussion at the spaCy forum.

We have now started working on a coref component native in spaCy v3 though. This won't be finished tomorrow, but we're making progress. We're going to reuse what we can from this library (with attribution, ofcourse) and make sure it trains and runs with native Thinc models :-)

@asterbini
Copy link

I was starting to look for a way to export the neuralcoref model in spacy2 as text, and try to load it in spacy3 :)
From your words this seems to be useless, thanks for letting me know.

@svlandeg
Copy link
Collaborator

I'm not a 100% sure though. If you have a working PyTorch model, that might operate with Thinc's new shim layers, but you'll have to write a custom component to wrap it in spaCy...

@asterbini
Copy link

asterbini commented Feb 26, 2021

But why the neuralcoref models should depend on the spacy3 models implementation? From a cursory glance to the NeuralCoref annotator class it seems to me that it reads only properties of the doc tokens that have not changed from v2 to v3. Then unless thinc 8 is changed radically wrt 7, (and by mapping Relu to ReLu and Linear to Affine) perhaps there is a way to convert the binary data model to thinc 8.
(and yet, as I am a newbie of thinc and neuralcoref an I am sure I could have missed something)

@KTRosenberg
Copy link

@asterbini : Due to the many internal changes between Thinc 7 and 8, and spaCy 2 and 3, I'm not sure this is going to be feasible. See also this discussion at the spaCy forum.

We have now started working on a coref component native in spaCy v3 though. This won't be finished tomorrow, but we're making progress. We're going to reuse what we can from this library (with attribution, ofcourse) and make sure it trains and runs with native Thinc models :-)

Out of curiosity since I'm trying to integrate Spacy and Neuralcoref into a time-sensitive project, is the move to Spacy 3 going to take on the order of a couple of weeks, a couple of months, or a year? I'm not familiar with the amount of work necessary to do something like this.

@svlandeg
Copy link
Collaborator

I'd estimate a month, a couple of months at most.

@KTRosenberg
Copy link

Thanks! In that case, I might downgrade temporarily.

@d5555
Copy link
Author

d5555 commented Mar 26, 2021

I'd estimate a month, a couple of months at most.

That's great news! I can't wait :)

@ZhuoruLin
Copy link

Wow. Can't wait to use neuralcoref with SpaCy V3!

@jazzdup
Copy link

jazzdup commented Apr 8, 2021

looking forward to this! thankyou!

@telmomenezes
Copy link

I'm staying with spaCy 2 until this comes out. Can't wait! Thanks for the great work!

@lauwauw
Copy link

lauwauw commented Apr 9, 2021

Also looking forward to this! Hoping to use this in my project with the en_core_web_trf pipeline, which is dependent on spaCy 3. Thanks indeed for the great work so far!

@chinmaydas96
Copy link

This is an awesome project try on, waiting for the spacy 3.0 compartibility.

@grigvardanyan
Copy link

Hi @svlandeg, do you have approximate estimation when neuralcoref for spacy3 will be released ?
Thanks in advance

@Ar9av
Copy link

Ar9av commented Jun 4, 2021

Following

@yutong-fei
Copy link

folloming

@seanswyi
Copy link

seanswyi commented Jun 5, 2021

Also following.

@svlandeg
Copy link
Collaborator

svlandeg commented Jun 10, 2021

Hi all, as a quick update that I also posted in a related thread.

While we (Explosion) originally intended to update neuralcoref in collaboration with HuggingFace (and @thomwolf had been very helpful with that), we've recently revamped spaCy to 3.0 and it turned out that quite a significant rewrite was necessary to support a native Thinc model in neuralcoref. Meanwhile also, research on coreference resolution had progressed further.

So long story short, we started working on a built-in coreference module that will be part of the core spaCy library. You can follow its development here: explosion/spaCy#7264. You'll see that we've been actively working on this. Most of the code is written, but it needs some more debugging and fine-tuning.

I can't speak for HuggingFace and what they intend to do with neuralcoref going forward and whether they're planning more updates, but either way this will always be a useful library for the older versions of spaCy :-)

@KTRosenberg
Copy link

Hi all, as a quick update that I also posted in a related thread.

While we (Explosion) originally intended to update neuralcoref in collaboration with HuggingFace (and @thomwolf had been very helpful with that), we've recently revamped spaCy to 3.0 and it turned out that quite a significant rewrite was necessary to support a native Thinc model in neuralcoref. Meanwhile also, research on coreference resolution had progressed further.

So long story short, we started working on a built-in coreference module that will be part of the core spaCy library. You can follow its development here: explosion/spaCy#7264. You'll see that we've been actively working on this. Most of the code is written, but it needs some more debugging and fine-tuning.

I can't speak for HuggingFace and what they intend to do with neuralcoref going forward and whether they're planning more updates, but either way this will always be a useful library for the older versions of spaCy :-)

That sounds good! Would a reasonable ETA be ~1 month for debugging and testing?

@svlandeg
Copy link
Collaborator

It's always difficult to provide these kind of estimates, as priorities sometimes shift, and we definitely want to "battle test" this new feature properly before we'll make it public.

@dogberto
Copy link

dogberto commented Jul 4, 2021

Awesome to hear this is happening! Thank you for putting the work into this. Please let us know when you think you have a release date :)

@KTRosenberg
Copy link

What is the status on this? Hopefully it's going well.

@nilesh
Copy link

nilesh commented Jul 13, 2021

Eagerly waiting for the core spacy3 integration. I can see that the pull request is out there and work in progress to merge. @KTRosenberg, you can track the progress here - explosion/spaCy#7264

@danyang-rainbow
Copy link

Looking forward to see the spacy 3.0 or even 3.1 integration!✔

@grigvardanyan
Copy link

@svlandeg do you have any news :) ?

@svlandeg
Copy link
Collaborator

svlandeg commented Jan 3, 2022

explosion/spaCy#7264 (comment) ;-)

@qhungngo
Copy link

Looking forward to using it in SpaCy 3.0

@polm
Copy link
Contributor

polm commented Oct 7, 2022

The new coreference component for spaCy 3 has been released.

@fakerybakery
Copy link

Hi @polm, personally, I think the neuralcoref library is much easier to use than the built-in coreference for SpaCy. Is it possible to make this library into a wrapper for the built-in coreference and preserve the same API?
Thank you!

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

No branches or pull requests