peft icon indicating copy to clipboard operation
peft copied to clipboard

Apply deprecated `evaluation_strategy`

Open muellerzr opened this issue 1 year ago • 5 comments

This PR applies the deprecated evaluation_strategy from https://github.com/huggingface/transformers/pull/30190.

TL;DR: evaluation_strategy -> eval_strategy

(Should go live in v4.41.0, but opening now so it's here when ready!)

muellerzr avatar Apr 18 '24 17:04 muellerzr

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Hmm, but this means that any older transformers version would result in an error, would it not? If we merge this PR, everyone trying to use those examples needs to upgrade to the latest transformers just to avoid getting a deprecation message. I think this is not worthwhile.

Ideally, we can revisit this PR in 6-12 months, when we can be reasonably sure that most users have moved to a new transformers version, or once transformers v4.46 is slated to be released.

BenjaminBossan avatar Apr 19 '24 09:04 BenjaminBossan

@BenjaminBossan will go live in v4.41.0 and stay until 4.46.0 as detailed in the thread (6 months)

muellerzr avatar Apr 19 '24 11:04 muellerzr

And yes, this PR is open for the future :)

muellerzr avatar Apr 19 '24 11:04 muellerzr

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

github-actions[bot] avatar Jun 11 '24 15:06 github-actions[bot]

I'm still withholding review to wait for

will go live in v4.41.0 and stay until 4.46.0 as detailed in the thread (6 months)

BenjaminBossan avatar Jun 11 '24 15:06 BenjaminBossan

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

github-actions[bot] avatar Jul 06 '24 15:07 github-actions[bot]

@muellerzr Has the time come now?

BenjaminBossan avatar Sep 13 '24 15:09 BenjaminBossan

@BenjaminBossan yes, time has come :)

muellerzr avatar Sep 13 '24 15:09 muellerzr