Skip to content

length penalty in reward function#14

Open
paulinebourigault wants to merge 4 commits into
mainfrom
pauline/length_penalty_in_reward
Open

length penalty in reward function#14
paulinebourigault wants to merge 4 commits into
mainfrom
pauline/length_penalty_in_reward

Conversation

@paulinebourigault

Copy link
Copy Markdown

The length penalty is calculated using the formula: ((5 + length) / 6) ^ alpha

This is based on the formula used in Google Neural Machine Translation paper [https://arxiv.org/pdf/1609.08144]
When alpha < 0, shorter sequences get higher rewards
When alpha > 0, longer sequences get higher rewards

Length penalties are applied to the final reward scores, after the primary reward calculation

@paulinebourigault

paulinebourigault commented Apr 25, 2025

Copy link
Copy Markdown
Author

example use in training script:
reward_model.length_penalty.enabled=True \
reward_model.length_penalty.alpha=-0.15 \
reward_model.length_penalty.min_length=3000 \
reward_model.length_penalty.max_length=4000 \

@paulinebourigault paulinebourigault changed the title length penaly in reward function length penalty in reward function Apr 25, 2025
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.

1 participant