Adds an option called "constrain_crf" to the SequenceLabelingModel class, if use_crf and constrain_crf are both True then it substitutes the CRF module with the CRFwithConstraints module that implements constrains for BIO and BIOES valid tag decoding.
Before submitting
[x] I've read and followed all steps in the Making a pull request
section of the CONTRIBUTING docs.
[x] I've updated or added any relevant docstrings following the syntax described in the create a branch and work section CONTRIBUTING docs.
[ ] If this PR fixes a bug, I've added a test that will fail without my fix.
[ ] If this PR adds a new feature, I've added tests that sufficiently cover my new functionality.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Fixes #21 .
Changes proposed in this pull request:
Before submitting
CONTRIBUTING
docs.CONTRIBUTING
docs.PR review
Anyone in the community is free to review the PR once the tests have passed. If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃