Upgrading transformers version to 4.51.3 to support recent models #175
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
num_items_in_batchtocompute_losssignatureeval_dataset=Nonetrainer.processing_classinstead oftrainer.tokenizer(it's depracated andtransformers==5removes it)Additionally it:
Related issues: #173 and #155
Before submitting
Other notes
I manually tested the new evaluation, with removed prediction_step from UnlearninTrainer, and it works the same.
When I tested unlearning, the overall unlearning trajectory is very similar, but there are some tiny changes to the logged training loss, likely due to a difference in how new transformers aggregates it. (Not sure though, but the differences were small.)