-
Notifications
You must be signed in to change notification settings - Fork 477
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
Comments
Yes! This is definitely on our road map :-) I'll actually pin this issue because I'm sure others are interested as well. |
Plus 1 |
After some rough paches I am able to start neuralcoref with spacy 3 but I obviously get the error |
@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 |
I was starting to look for a way to export the neuralcoref model in spacy2 as text, and try to load it in spacy3 :) |
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... |
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. |
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. |
I'd estimate a month, a couple of months at most. |
Thanks! In that case, I might downgrade temporarily. |
That's great news! I can't wait :) |
Wow. Can't wait to use neuralcoref with SpaCy V3! |
looking forward to this! thankyou! |
I'm staying with spaCy 2 until this comes out. Can't wait! Thanks for the great work! |
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! |
This is an awesome project try on, waiting for the spacy 3.0 compartibility. |
Hi @svlandeg, do you have approximate estimation when neuralcoref for spacy3 will be released ? |
Following |
folloming |
Also following. |
Hi all, as a quick update that I also posted in a related thread. While we (Explosion) originally intended to update 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 |
That sounds good! Would a reasonable ETA be ~1 month for debugging and testing? |
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. |
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 :) |
What is the status on this? Hopefully it's going well. |
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 |
Looking forward to see the spacy 3.0 or even 3.1 integration!✔ |
@svlandeg do you have any news :) ? |
Looking forward to using it in SpaCy 3.0 |
The new coreference component for spaCy 3 has been released. |
Any Update on this one? Spacy has been keeping coref feature as experimental for 2 years now and there is not enough documentation to get it work. Would be nice to get this updated to v3 version |
Is there going to be neuralcoref compatible with spaCy 3.0 ?
The text was updated successfully, but these errors were encountered: