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

Add remote checkpoint support to ray tune post search evaluation #1646

Merged
merged 1 commit into from
Jan 5, 2022

Conversation

amholler
Copy link
Collaborator

@amholler amholler commented Jan 5, 2022

Recently Ludwig was updated to perform model evaluation after the Ray Tune search was complete,
for any trials that were terminated before their model evaluation step was performed. This update did not
handle the case where the checkpoint was remote, e.g., when output_directory was set to an s3 bucket.
This change set adds that handling. Will create a cross-port of this change set for master.

@github-actions
Copy link

github-actions bot commented Jan 5, 2022

Unit Test Results

       3 files   -     3         3 suites   - 3   1h 45m 56s ⏱️ - 1h 2m 13s
1 135 tests  -   81  1 119 ✔️  -   73  16 💤  -   8  0 ±0 
3 405 runs   - 243  3 357 ✔️  - 219  48 💤  - 24  0 ±0 

Results for commit accd0d9. ± Comparison against base commit 5192a2a.

@justinxzhao justinxzhao merged commit 9bf0272 into ludwig-ai:tf-legacy Jan 5, 2022
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