allennlp icon indicating copy to clipboard operation
allennlp copied to clipboard

Add allowed_transitions in __init__.py of crf mod.

Open iamlockelightning opened this issue 3 years ago • 0 comments

Fix ImportError: from allennlp.modules.conditional_random_field import allowed_transitions ImportError: cannot import name 'allowed_transitions' from 'allennlp.modules.conditional_random_field'

Fixes #5695.

Changes proposed in this pull request:

  • Fix ImportError of allowed_transitions

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 Writing docstrings section of the CONTRIBUTING docs.
  • [x] If this PR fixes a bug, I've added a test that will fail without my fix.
  • [x] If this PR adds a new feature, I've added tests that sufficiently cover my new functionality.

After submitting

  • [ ] All GitHub Actions jobs for my pull request have passed.
  • [ ] codecov/patch reports high test coverage (at least 90%). You can find this under the "Actions" tab of the pull request once the other checks have finished.

iamlockelightning avatar Aug 01 '22 13:08 iamlockelightning

I finished this in #5706. Thank you!

dirkgr avatar Aug 24 '22 22:08 dirkgr