Skip to content

Implement gsutil download for GCS URLs#901

Merged
jimmyt857 merged 5 commits intoPhysical-Intelligence:mainfrom
psimit:900-gcloud-download-defaults-to-authentication-even-for-public-assets
Mar 13, 2026
Merged

Implement gsutil download for GCS URLs#901
jimmyt857 merged 5 commits intoPhysical-Intelligence:mainfrom
psimit:900-gcloud-download-defaults-to-authentication-even-for-public-assets

Conversation

@psimit
Copy link
Contributor

@psimit psimit commented Mar 12, 2026

Add a new function to download files using gsutil for GCS URLs.

Add a new function to download files using gsutil for GCS URLs.
@psimit
Copy link
Contributor Author

psimit commented Mar 12, 2026

This addresses Gcloud download defaults to authentication even for public assets (#900)

psimit added 2 commits March 12, 2026 17:27
- Fallback to default download method if gsutil is not available
- Update _download_gsutil to accept additional kwargs for flexibility in downloading files.
@jimmyt857 jimmyt857 requested a review from kvablack as a code owner March 12, 2026 16:35
@jimmyt857 jimmyt857 enabled auto-merge March 12, 2026 16:35
@jimmyt857
Copy link
Contributor

Thanks!

auto-merge was automatically disabled March 13, 2026 05:13

Head branch was pushed to by a user without write access

@psimit psimit force-pushed the 900-gcloud-download-defaults-to-authentication-even-for-public-assets branch from 16d8bf9 to 2601fa2 Compare March 13, 2026 05:26
@jimmyt857 jimmyt857 enabled auto-merge March 13, 2026 15:33
@jimmyt857 jimmyt857 merged commit e6b0441 into Physical-Intelligence:main Mar 13, 2026
2 checks passed
@psimit psimit deleted the 900-gcloud-download-defaults-to-authentication-even-for-public-assets branch March 13, 2026 18:23
LPSlv pushed a commit to LPSlv/openpi that referenced this pull request Mar 14, 2026
Add a new function to download files using gsutil for GCS URLs.
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.

2 participants