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.

Sign up for GitHub

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

Jump to bottom

[AIR][Tune] Make SyncConfig(sync_artifacts=False) work fully (for all syncing methods) #32783

Closed
2 tasks
justinvyu opened this issue Feb 23, 2023 · 1 comment
Closed
2 tasks

[AIR][Tune] Make SyncConfig(sync_artifacts=False) work fully (for all syncing methods) #32783

justinvyu opened this issue Feb 23, 2023 · 1 comment
Labels
P2 Important issue, but not time-critical ray-team-created Ray Team created tune Tune-related issues

Comments

@justinvyu
Copy link
Contributor

justinvyu commented Feb 23, 2023 •
edited
Loading

#32334 introduced a SyncConfig(sync_artifacts) flag.

  • Cloud syncing only disables artifact uploading for worker trials. Driver trials will still upload their artifacts.
  • Default object store syncing will still ship all artifacts to the driver. This flag doesn't apply to that case at all.
@justinvyu justinvyu added tune Tune-related issues P2 Important issue, but not time-critical air labels Feb 23, 2023
@justinvyu justinvyu mentioned this issue Feb 23, 2023
Merged
7 tasks
@matthewdeng matthewdeng added the ray-team-created Ray Team created label Apr 3, 2023
@anyscalesam anyscalesam removed the air label Oct 28, 2023
@justinvyu
Copy link
Contributor Author

Syncing paths have been unified -- sync_artifacts now works for all storage types.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P2 Important issue, but not time-critical ray-team-created Ray Team created tune Tune-related issues
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@justinvyu @matthewdeng @anyscalesam