Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Migrate to Hydra 1.0 #536

Closed

Conversation

mannatsingh
Copy link
Contributor

Summary:

  • Migrated to Hydra 1.0
  • We need to figure out what the best hierarchy is for Classy given Hydra's new capabilities since the original decision might not be the best now
    • I tried out package directives as well and they work well
    • Leaving this for the future though

Documenting the API changes here -

pip install .
pip install hydra-core==1.0.0rc1 # to be replaced with final version
classy_project my_hydra_project

# some example runs with the new API
# note that since task, optimizer and param_scheduler are used in args.yaml, they will not need a + prefix in the command

./my_hydra_project/classy_train.py +model=resnet_50
./my_hydra_project/classy_train.py +loss=cross_entropy +model=resnet50

Differential Revision: D21916837

@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 6, 2020
@facebook-github-bot
Copy link
Contributor

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

mannatsingh added a commit to mannatsingh/ClassyVision that referenced this pull request Jun 6, 2020
Summary:
Pull Request resolved: facebookresearch#536

- Migrated to Hydra 1.0
- We need to figure out what the best hierarchy is for Classy given Hydra's new capabilities since the original decision might not be the best now
  - I tried out package directives as well and they work well
  - Leaving this for the future though

Documenting the API changes here -
```
pip install .
pip install hydra-core==1.0.0rc1 # to be replaced with final version
classy_project my_hydra_project

# some example runs with the new API

./my_hydra_project/classy_train.py model=resnet_50
./my_hydra_project/classy_train.py loss=label_smoothing_cross_entropy model=resnet50
./my_hydra_project/classy_train.py loss=label_smoothing_cross_entropy config=resnet50_synthetic
```

Reviewed By: omry

Differential Revision: D21916837

fbshipit-source-id: 2c1ff0cbafb1e5e3b9d396ced0484d77b00acdd8
@facebook-github-bot
Copy link
Contributor

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

mannatsingh added a commit to mannatsingh/ClassyVision that referenced this pull request Jun 8, 2020
Summary:
Pull Request resolved: facebookresearch#536

- Migrated to Hydra 1.0
- We need to figure out what the best hierarchy is for Classy given Hydra's new capabilities since the original decision might not be the best now
  - I tried out package directives as well and they work well
  - Leaving this for the future though

Documenting the API changes here -
```
pip install .
pip install hydra-core==1.0.0rc1 # to be replaced with final version
classy_project my_hydra_project

# some example runs with the new API

./my_hydra_project/classy_train.py model=resnet_50
./my_hydra_project/classy_train.py loss=label_smoothing_cross_entropy model=resnet50
./my_hydra_project/classy_train.py loss=label_smoothing_cross_entropy config=resnet50_synthetic
```

Reviewed By: omry

Differential Revision: D21916837

fbshipit-source-id: c96966633fff31a87b79a8f97ef4385f96e1eb60
@facebook-github-bot
Copy link
Contributor

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

Summary:
Pull Request resolved: facebookresearch#536

- Migrated to Hydra 1.0
- We need to figure out what the best hierarchy is for Classy given Hydra's new capabilities since the original decision might not be the best now
  - I tried out package directives as well and they work well
  - Leaving this for the future though

Documenting the API changes here -
```
pip install .
pip install hydra-core==1.0.0rc1 # to be replaced with final version
classy_project my_hydra_project

# some example runs with the new API

./my_hydra_project/classy_train.py model=resnet_50
./my_hydra_project/classy_train.py loss=label_smoothing_cross_entropy model=resnet50
./my_hydra_project/classy_train.py loss=label_smoothing_cross_entropy config=resnet50_synthetic
```

Reviewed By: omry

Differential Revision: D21916837

fbshipit-source-id: a4c0b7e9a3cda98535aced1a6bfa9dc8b6a030ca
@facebook-github-bot
Copy link
Contributor

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

facebook-github-bot pushed a commit that referenced this pull request Jun 9, 2020
Summary:
Pull Request resolved: #536

- Migrated to Hydra 1.0
- We need to figure out what the best hierarchy is for Classy given Hydra's new capabilities since the original decision might not be the best now
  - I tried out package directives as well and they work well
  - Leaving this for the future though

Documenting the API changes here -
```
pip install .
pip install hydra-core==1.0.0rc1 # to be replaced with final version
classy_project my_hydra_project

# some example runs with the new API

./my_hydra_project/classy_train.py model=resnet_50
./my_hydra_project/classy_train.py loss=label_smoothing_cross_entropy model=resnet50
./my_hydra_project/classy_train.py loss=label_smoothing_cross_entropy config=resnet50_synthetic
```

Reviewed By: omry

Differential Revision: D21916837

fbshipit-source-id: 9e75b85a1155a9b6b620d72f8d39aa5d02171e78
@mannatsingh mannatsingh deleted the export-D21916837 branch June 29, 2020 21:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants