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.

Sign up for GitHub

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

Remove sentences access from spans property. #305

Open

dafajon wants to merge 1 commit into develop from feature/spans_fix

Contributor

dafajon commented Dec 21, 2021

Calling spans of a document requires access to sents which loads sbd. However training an sbd model requires access to spans. This inflicts a circular dependency which is redundant and might cause issues if an sbd model is trained from scratch.


Remove sentneces access from spans property.

d6ab6aa

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment