Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Split hp search methods #6857

Merged
merged 2 commits into from
Aug 31, 2020
Merged

Split hp search methods #6857

merged 2 commits into from
Aug 31, 2020

Conversation

sgugger
Copy link
Collaborator

@sgugger sgugger commented Aug 31, 2020

Follow-up from #6747.

Cleanly separates the two backend-specific code (optuna vs Ray) with some small code duplication in the objective function.

@sgugger sgugger merged commit a59bcef into master Aug 31, 2020
@sgugger sgugger deleted the split_hp_search_methods branch August 31, 2020 19:16
stas00 pushed a commit to stas00/transformers that referenced this pull request Sep 1, 2020
* Split the run_hp_search by backend

* Unused import
@LysandreJik
Copy link
Member

Cool!

Zigur pushed a commit to Zigur/transformers that referenced this pull request Oct 26, 2020
* Split the run_hp_search by backend

* Unused import
fabiocapsouza pushed a commit to fabiocapsouza/transformers that referenced this pull request Nov 15, 2020
* Split the run_hp_search by backend

* Unused import
fabiocapsouza added a commit to fabiocapsouza/transformers that referenced this pull request Nov 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants