search

grammarly / gector

Official implementation of the papers "GECToR – Grammatical Error Correction: Tag, Not Rewrite" (BEA-20) and "Text Simplification by Tagging" (BEA-21)
Apache License 2.0
891 stars 216 forks source link

Bump version of dependencies #153

Closed ierezell closed 2 years ago

ierezell commented 2 years ago

Hi,

First thanks a lot for this great work ! It's really nice to have a SOTA model available. I started to play with the model and everything works perfectly.

However, it's impossible to integrate it with newer codebases, anything that depends on spacy/flair/allennlp would get conflict in dependencies as allennlp 0.8.4 (2.9.2 today) or conllu 0.11 (4.4.1 today) are quite old...

It would be nice to update them so we can use this code in actual projects.

Thanks a lot in advance, Have a great day.

skurzhanskyi commented 2 years ago

Hi @Ierezell It's good to hear that you found our repository useful. Regarding the allennlp update, we don't have plans to do it, for now. This step would require rewriting the scripts significantly. And we've already updated the transformers library. If you would like to prepare PR with the updates, we would be happy to review it and merge.