Fix: tp size wouldn't read from env by S1ro1 · Pull Request #3716 · huggingface/accelerate
Conversation
With change in #3700 where we default to None, passing 1 as tp_size would result in env vars not being read.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch. thanks!
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks ! nice catch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters