-
Notifications
You must be signed in to change notification settings - Fork 757
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
feat: use yatai proxy to upload/download bentos/models #2832
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2832 +/- ##
==========================================
- Coverage 70.93% 70.69% -0.25%
==========================================
Files 103 103
Lines 9334 9380 +46
==========================================
+ Hits 6621 6631 +10
- Misses 2713 2749 +36
|
@yetone Do we want to make s3 presigned URL an option for user to enable? we may need that for backwards compatible as well |
0e2b8b4
to
a83684b
Compare
Hello @yetone, Thanks for updating this PR. There are currently no PEP 8 issues detected in this PR. Cheers! 🍻 Comment last updated at 2022-08-17 10:35:57 UTC |
bd634ee
to
a6dec06
Compare
f7c55ab
to
974b812
Compare
@parano Forward compatibility is complete, please review and merge |
cpu=r.resource_config.cpu, | ||
nvidia_gpu=r.resource_config.nvidia_gpu, | ||
custom_resources=r.resource_config.custom_resources, | ||
cpu=r.resource_config.get("cpu"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope this type-level refactoring needs to be synchronized to all the places where the type is used, and it feels like we need to add all the type checks to our ci, otherwise, it's a nullity
974b812
to
301a64c
Compare
376b627
to
183dc75
Compare
183dc75
to
f95765c
Compare
No description provided.