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

Hydra plugin now allow to search the configuration in the current path. #161

Closed

Conversation

QuentinDuval
Copy link
Contributor

This is useful in the following test case: running VISSL from another directory on which some configurations are stored separately from the VISSL repository (for instance, in order to create configurations that should not end up being committed in VISSL by mistake, or configurations that are specific to another client repository).

…current path. It hurts the following test case: running VISSL from another directory on which configurations is stored, which is useful in order to create configurations that should not end up in VISSL
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 29, 2021
@QuentinDuval QuentinDuval requested a review from prigoyal January 29, 2021 16:35
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@QuentinDuval has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

facebook-github-bot pushed a commit that referenced this pull request Jan 29, 2021
…h. (#161)

Summary:
This is useful in the following test case: running VISSL from another directory on which some configurations are stored separately from the VISSL repository (for instance, in order to create configurations that should not end up being committed in VISSL by mistake, or configurations that are specific to another client repository).

Pull Request resolved: #161

Reviewed By: prigoyal

Differential Revision: D26149625

Pulled By: QuentinDuval

fbshipit-source-id: 72c8705d11fd1afd1c6cabfef7678944472ddc52
@QuentinDuval QuentinDuval deleted the hydra_search_path branch February 8, 2021 23:05
QuentinDuval added a commit to QuentinDuval/vissl that referenced this pull request Jul 6, 2021
Summary: Pull Request resolved: fairinternal/ssl_scaling#161

Differential Revision: D29458822

Pulled By: QuentinDuval

fbshipit-source-id: 8fb6e26c13f9077de3ef59089a7dfe12329961a7
QuentinDuval added a commit to QuentinDuval/vissl that referenced this pull request Jul 6, 2021
Summary: Pull Request resolved: fairinternal/ssl_scaling#161

Differential Revision: D29458822

Pulled By: QuentinDuval

fbshipit-source-id: 39bc02ada50e6a4c7937aa723ba7cfaf7962235c
QuentinDuval added a commit to QuentinDuval/vissl that referenced this pull request Jul 6, 2021
Summary: Pull Request resolved: fairinternal/ssl_scaling#161

Differential Revision: D29458822

Pulled By: QuentinDuval

fbshipit-source-id: 5162af9f279b9951648efc510d2fccd050a2c2bc
QuentinDuval added a commit to QuentinDuval/vissl that referenced this pull request Jul 7, 2021
Summary: Pull Request resolved: fairinternal/ssl_scaling#161

Differential Revision: D29458822

Pulled By: QuentinDuval

fbshipit-source-id: fc2438d8c82f5776961dd04029bd2c5722c78398
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants