-
Notifications
You must be signed in to change notification settings - Fork 6.7k
Closed
Labels
bugSomething isn't workingSomething isn't working
Description
Describe the bug
Across several files, a bool statement is wrongly constructed IMO
One example:
| CTX = init_empty_weights if is_accelerate_available else nullcontext |
is_accelerate_available itself, it should check for the truthiness of its return value I believe. So parentheses are needed, to call this function.
Proposed fix:
CTX = init_empty_weights if is_accelerate_available() else nullcontext Reproduction
see
| CTX = init_empty_weights if is_accelerate_available else nullcontext |
Logs
No response
System Info
not relevant
Who can help?
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working