You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The change should be implemented in a backwards compatible manner, so that passing the instantiated Accelerator keeps on working.
The reason for this change is not so much that we want to, for instance, grid search on the Accelerator parameters, which makes little sense. Mainly, it would be for consistency, so that the Accelerator can be instantiated and configured in the same way as other components like the module and optimizer.
The text was updated successfully, but these errors were encountered:
Right now, when using
AccelerateMixin
, the instantiatedAccelerator
instance needs to be passed:This new feature should allow to pass the
Accelerator
class directly, with possible extra arguments using the__
notation:The change should be implemented in a backwards compatible manner, so that passing the instantiated
Accelerator
keeps on working.The reason for this change is not so much that we want to, for instance, grid search on the
Accelerator
parameters, which makes little sense. Mainly, it would be for consistency, so that theAccelerator
can be instantiated and configured in the same way as other components like the module and optimizer.The text was updated successfully, but these errors were encountered: