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

Keep search_alg type in dict to record in hyperopt_statistics.json #1626

Merged
merged 1 commit into from
Dec 26, 2021

Conversation

amholler
Copy link
Collaborator

As titled. Without this change, the search_alg type that was used is not recorded in hyperopt_statistics.json.
[It is logged correctly in the run output, which is recorded before this pop op, but users don't typically retain that output.]

@github-actions
Copy link

Unit Test Results

       6 files  ±0         6 suites  ±0   2h 35m 25s ⏱️ - 25m 58s
1 216 tests ±0  1 192 ✔️ ±0  24 💤 ±0  0 ±0 
3 648 runs  ±0  3 576 ✔️ ±0  72 💤 ±0  0 ±0 

Results for commit 1560f5e. ± Comparison against base commit 20a5847.

@justinxzhao justinxzhao merged commit 2945168 into ludwig-ai:tf-legacy Dec 26, 2021
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.

3 participants