Skip to content

Fix: rename 'eval_strategy' to 'evaluation_strategy' in TrainingArgum…#39109

Open
trevorhughdavis wants to merge 1 commit into
huggingface:mainfrom
trevorhughdavis:fix-evaluation-strategy-param
Open

Fix: rename 'eval_strategy' to 'evaluation_strategy' in TrainingArgum…#39109
trevorhughdavis wants to merge 1 commit into
huggingface:mainfrom
trevorhughdavis:fix-evaluation-strategy-param

Conversation

@trevorhughdavis

Copy link
Copy Markdown

🐛 Bugfix: Rename eval_strategyevaluation_strategy in TrainingArguments

This PR corrects an API mismatch in the TrainingArguments dataclass where the constructor expected eval_strategy, but Trainer, config files, and CLI use evaluation_strategy.

✅ Changes

  • Renames the field eval_strategyevaluation_strategy
  • Updates all internal logic and references
  • Corrects docstrings and argument descriptions
  • Validated via inspect.signature(TrainingArguments.__init__)
  • Tested with full reinstallation and cache invalidation

🧪 Reproduction (Before Fix)

TrainingArguments(evaluation_strategy="epoch")
# ❌ TypeError: got an unexpected keyword argument 'evaluation_strategy'

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

Successfully merging this pull request may close these issues.

2 participants