We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
trainer
When training a Tokenizer, we must provide a Trainer:
Tokenizer
Trainer
trainer = BpeTrainer(...) tokenizer.train(files=[ ... ], trainer=trainer)
This is true even if we instantiate the BpeTrainer without changing any defaults.
BpeTrainer
Makes the Trainer optional:
tokenizer.train(files=[ ... ])
The Model should be able to return a default trainer:
Model
trainer = model.get_trainer()
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
Current state
When training a
Tokenizer
, we must provide aTrainer
:This is true even if we instantiate the
BpeTrainer
without changing any defaults.Goal
Makes the
Trainer
optional:How
The
Model
should be able to return a default trainer:The text was updated successfully, but these errors were encountered: