This repository was archived by the owner on Jan 22, 2025. It is now read-only.
make builtin runner's get_default_cfg to use config updater#272
Open
wat3rBro wants to merge 2 commits intofacebookresearch:mainfrom
Open
make builtin runner's get_default_cfg to use config updater#272wat3rBro wants to merge 2 commits intofacebookresearch:mainfrom
wat3rBro wants to merge 2 commits intofacebookresearch:mainfrom
Conversation
Summary: - set `get_default_cfg = None` to opt-in this new feature. - support config with `_DEFAULTS_` when loading config file. Note that we don't check explicitly `"_DEFAULTS_" in config file` but check `runner.get_default_cfg == None`, this is because runner having `get_default_cfg` and config having `_DEFAULTS_` should be mutually exclusive, and `load_full_config_from_file` can raise proper error if `_DEFAULTS_` is missing. - we also need to save `_DEFAULTS_` in the diff config. Differential Revision: D36868581 fbshipit-source-id: 4cc2d516c45d8f161f37a8c6f2a8146cc03e3462
Differential Revision: D36463900 fbshipit-source-id: 4d4b5ee0bd9cfece35e1ed6f5090fe828d4b71ff
facebook-github-bot
added
CLA Signed
Contributor
|
This pull request was exported from Phabricator. Differential Revision: D36463900 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Differential Revision: D36463900