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

Add unit tests and regression tests and bump Python to 3.8 #169

Closed damien2012eng closed 1 year ago

damien2012eng commented 1 year ago

Hello,

As discussed before, our team has been working on updating GECToR dependencies to the latest versions (Inference only). This is the first PR, a couple more PRs are coming soon. Link to the successful run: here We also checked the impact for the training process. No breaks were identified.

First PR: Add unit tests and regression test. Additionally, bump Python version. Second PR: Write a predictor class to mimic the GEC_model. Third PR: Unpin all the dependency requirements

How to review:

Our plan is to submit two more PRs. Suggestions and comments are welcome!

damien2012eng commented 1 year ago

Please ignore this PR. Wrong branch is used.