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_seq_length and max_seq_len confusion #29

Open
davidqqq opened this issue Jan 16, 2024 · 1 comment
Open

max_seq_length and max_seq_len confusion #29

davidqqq opened this issue Jan 16, 2024 · 1 comment

Comments

@davidqqq
Copy link

davidqqq commented Jan 16, 2024

Hi team!

Are these two max_seq_length and max_seq_len supposed to be the same parameter?

TASK_EXTRA="--max_seq_len 256 --first_sent_limit 240"

--max_seq_length 128

Only max_seq_length is referenced in script. Not sure if its a bug.

@gaotianyu1350
Copy link
Member

Hi,

Sorry for the confusion! Here max_seq_length is the correct (and full) name. But max_seq_len also works as the script will automatically map it to max_seq_length as a prefix.

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

No branches or pull requests

2 participants