-
Notifications
You must be signed in to change notification settings - Fork 260
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
Improvement: accept int value for Resources #2196
Conversation
Thank you for opening this pull request! 🙌 These tips will help get your PR across the finish line:
|
f3f0c0a
to
42159b5
Compare
f210f77
to
7b03b8d
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2196 +/- ##
==========================================
- Coverage 83.04% 80.59% -2.45%
==========================================
Files 324 180 -144
Lines 24861 17856 -7005
Branches 3547 3483 -64
==========================================
- Hits 20645 14391 -6254
+ Misses 3591 2843 -748
+ Partials 625 622 -3 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Ryan Lo <wenchih@apache.org>
d974e01
to
e535b34
Compare
Signed-off-by: Ryan Lo <wenchih@apache.org>
ea42aca
to
fa90d9c
Compare
Signed-off-by: Ryan Lo <wenchih@apache.org>
861ccf1
to
3853409
Compare
Signed-off-by: Ryan Lo <wenchih@apache.org>
Can you merge master ? |
Congrats on merging your first pull request! 🎉 |
Signed-off-by: Ryan Lo <wenchih@apache.org> Co-authored-by: Kevin Su <pingsutw@gmail.com>
Signed-off-by: Ryan Lo <wenchih@apache.org> Co-authored-by: Kevin Su <pingsutw@gmail.com>
Signed-off-by: Ryan Lo <wenchih@apache.org> Co-authored-by: Kevin Su <pingsutw@gmail.com> Signed-off-by: Jan Fiedler <jan@union.ai>
Tracking issue
Closes flyteorg/flyte #4691
Why are the changes needed?
Make str and int available for mem, gpu and ephemeral_storage.
Make str, int, float available for cpu.
What changes were proposed in this pull request?
Update Resources class in flytekit/core/resources.py
How was this patch tested?
Setup process
Screenshots
Check all the applicable boxes
Related PRs
Docs link