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 suffix to download_s3_folder args #3113

Merged
merged 1 commit into from
Apr 6, 2023

Conversation

Innixma
Copy link
Contributor

@Innixma Innixma commented Apr 6, 2023

Issue #, if available:

Description of changes:

  • Add suffix arg to download_s3_folder args
  • This allows filtering of the files to download based on suffix, which is useful for ZS-HPO project.
  • Follow-up PR should include the remaining args from list_bucket_prefix_suffix_contains_s3, as well as add unit tests for list_bucket_prefix_suffix_contains_s3.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Innixma Innixma added this to the 0.8 Release milestone Apr 6, 2023
@Innixma Innixma requested a review from yinweisu April 6, 2023 02:54
Copy link
Collaborator

@yinweisu yinweisu left a comment

Choose a reason for hiding this comment

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

LGTM!

@Innixma Innixma merged commit d8897a9 into autogluon:master Apr 6, 2023
21 of 22 checks passed
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

2 participants