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

max_length in README is not correct? #16

Closed
caterpillar0111 opened this issue Mar 5, 2022 · 1 comment
Closed

max_length in README is not correct? #16

caterpillar0111 opened this issue Mar 5, 2022 · 1 comment
Assignees
Labels
Priority: Trivial least priority Status: 3-Completed finished Type: Bug not work as expected

Comments

@caterpillar0111
Copy link

Hi, thanks for your great work.
I found a tiny error for your example, when execute the code
ws = ws_driver(text, batch_size=256, max_length=512)
It would show the error message is that
"AssertionError: Sequence length is longer than the maximum sequence length for this model (512 > 510)."
Set the max_length lower than 510 can fix this.
Without that, everything is fine. It's a excellent and convenience tool for extract information from data.

@emfomy emfomy self-assigned this Mar 7, 2022
@emfomy emfomy added Priority: Medium third priority Status: 1-Assigned assigned an assignee Type: Bug not work as expected Priority: Trivial least priority and removed Priority: Medium third priority labels Mar 7, 2022
@emfomy
Copy link
Member

emfomy commented Mar 7, 2022

The example is wrong. Fixed in v0.2.8.

@emfomy emfomy added Status: 3-Completed finished and removed Status: 1-Assigned assigned an assignee labels Mar 7, 2022
@emfomy emfomy closed this as completed Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Trivial least priority Status: 3-Completed finished Type: Bug not work as expected
Projects
None yet
Development

No branches or pull requests

2 participants