Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use spaCy v2.3.0 or later to use HocrVisualParser #528

Merged
merged 2 commits into from
Oct 26, 2020

Conversation

HiromuHota
Copy link
Contributor

@HiromuHota HiromuHota commented Oct 23, 2020

Description of the problems or issues

Is your pull request related to a problem? Please describe.

See #527

Does your pull request fix any issue.

Fix #527

Description of the proposed changes

Raise an ImportError when spacy < 2.3.0 to make sure spacy v2.3.0 or later is used for HocrVisualParser

Test plan

Introduce two consecutive single quotes '' to one of the example hOCR files.

Checklist

  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have updated the CHANGELOG.rst accordingly.

Copy link
Collaborator

@senwu senwu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@senwu senwu merged commit bd5b1fe into HazyResearch:master Oct 26, 2020
@HiromuHota
Copy link
Contributor Author

@senwu Thanks!

@HiromuHota HiromuHota deleted the fix/527 branch October 26, 2020 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tokens not aligned error when spacy < 2.3.0
2 participants