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

Upstream/1 add unit tests and regression test #171

Open damien2012eng opened 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:

Unit test. pytest -v tests Regression test. python regression_tests/test_gector_roberta.py Our plan is to submit two more PRs. Suggestions and comments are welcome!