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
{{ message }}
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.
Currently, the training backends are hard coded in hydra's code. There are a few drawbacks with this approach:
All the dependencies for all the training platforms must be installed even though one might not need some of them. otherwise the imports on top of cli.py will fail. (I will elaborate on this issue in a separate feature request issue)
The user is not able to easily add a new training backend or customize the existing ones (by subclassing them or AbstractPlatform)
I propose to have a code that dynamically discovers the AbstractPlatform subclasses in runtime and registers them with hydra. It will get rid of switch-case-like if statements such as the one above and https://github.com/georgianpartners/hydra/blob/7c733fd0eb7f9a9081ecdd2b5e3a5db2bdc282d4/hydra/cli.py#L74-L77
This discovery interface should also provide the users with an appropriate way (such as a method or a decorator) for users to register and load their custom training platforms manually.
--
I went ahead of myself and started working on a PR before opening the issue. I'll submit it once it's finalized.
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Currently, the training backends are hard coded in hydra's code. There are a few drawbacks with this approach:
cli.py
will fail. (I will elaborate on this issue in a separate feature request issue)AbstractPlatform
)I propose to have a code that dynamically discovers the
AbstractPlatform
subclasses in runtime and registers them with hydra. It will get rid of switch-case-like if statements such as the one above and https://github.com/georgianpartners/hydra/blob/7c733fd0eb7f9a9081ecdd2b5e3a5db2bdc282d4/hydra/cli.py#L74-L77This discovery interface should also provide the users with an appropriate way (such as a method or a decorator) for users to register and load their custom training platforms manually.
--
I went ahead of myself and started working on a PR before opening the issue. I'll submit it once it's finalized.
The text was updated successfully, but these errors were encountered: