From 2594bd8b73347e26dc7e07c2519d29de380fb46e Mon Sep 17 00:00:00 2001 From: Roman Kalyakin Date: Fri, 20 Nov 2020 19:33:54 +0100 Subject: [PATCH] moved temperature wrapper before topP/topK (#8686) --- src/transformers/generation_utils.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/transformers/generation_utils.py b/src/transformers/generation_utils.py index 6f99460ca5..918d2d1e10 100644 --- a/src/transformers/generation_utils.py +++ b/src/transformers/generation_utils.py @@ -244,12 +244,12 @@ class GenerationMixin: # the following idea is largely copied from this PR: https://github.com/huggingface/transformers/pull/5420/files # all samplers can be found in `generation_utils_samplers.py` + if temperature is not None and temperature != 1.0: + warpers.append(TemperatureLogitsWarper(temperature)) if top_k is not None and top_k != 0: warpers.append(TopKLogitsWarper(top_k=top_k, min_tokens_to_keep=(2 if num_beams > 1 else 1))) if top_p is not None and top_p < 1.0: warpers.append(TopPLogitsWarper(top_p=top_p, min_tokens_to_keep=(2 if num_beams > 1 else 1))) - if temperature is not None and temperature != 1.0: - warpers.append(TemperatureLogitsWarper(temperature)) return warpers def _get_logits_processor(