Skip to content

Conversation

aivanou
Copy link
Contributor

@aivanou aivanou commented Jun 23, 2021

Summary:
Move component search registration into a separate module and share it across runner and torch_cli
Introduce new format for [torchx.components] :

foo = my_project.some_module.bar

Where my_project.some_module.bar is a python module. The components will be recursively found in the dir for my_project.some_module.bar.__file__ file.

Reviewed By: kiukchung

Differential Revision: D29202878

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Jun 23, 2021
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D29202878

aivanou and others added 2 commits June 23, 2021 17:41
Summary:
Pull Request resolved: meta-pytorch#89

Make torchx compatible with python 3.7

Differential Revision: D29348727

fbshipit-source-id: f033d13559afed83473c6e9caf50ce9c51f4ff01
Summary:
Pull Request resolved: meta-pytorch#88

Move component search registration into a separate module and share it across runner and torch_cli
Introduce new format for ``[torchx.components]`` :

    foo = my_project.some_module.bar

Where ``my_project.some_module.bar`` is a python module. The components will be recursively found in the dir for ``my_project.some_module.bar.__file__`` file.

Reviewed By: kiukchung

Differential Revision: D29202878

fbshipit-source-id: ac17e6e597ca3fc92e916e9729d6c16e5c045cb3
@aivanou aivanou force-pushed the export-D29202878 branch from a1788c7 to 66103ff Compare June 24, 2021 00:41
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D29202878

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 76bb0c0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants