RL4LMs icon indicating copy to clipboard operation
RL4LMs copied to clipboard

Published 20 hours ago • verified publisher icon allenai

100% likely that two function parameters have been merged by accident

Open JulesGM opened this issue 2 years ago • 1 comments

https://github.com/allenai/RL4LMs/blob/main/rl4lms/envs/text_generation/policy/seq2seq_policy.py#L263

Screen Shot 2022-11-29 at 6 05 29 PM

JulesGM avatar Nov 29 '22 23:11 JulesGM

Oh right. This seems like a merging issue. The good thing is that argument is unused/overridden at the moment. That is why it still works without any run-time errors.

rajcscw avatar Dec 01 '22 08:12 rajcscw