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

Refactor resource limit in pytorch/tensorflow plugin #724

Merged
merged 3 commits into from
Oct 29, 2021
Merged

Conversation

yindia
Copy link
Contributor

@yindia yindia commented Oct 28, 2021

TL;DR

Refactor resource limit in pytorch/tensorflow plugin

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

How did you fix the bug, make the feature etc. Link to any design docs etc

Tracking Issue

flyteorg/flyte#1758

Follow-up issue

NA
OR
https://github.com/lyft/flyte/issues/

@codecov
Copy link

codecov bot commented Oct 28, 2021

Codecov Report

Merging #724 (7351644) into master (87131a0) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #724   +/-   ##
=======================================
  Coverage   85.78%   85.78%           
=======================================
  Files         362      362           
  Lines       30007    30007           
  Branches     2444     2444           
=======================================
  Hits        25743    25743           
  Misses       3621     3621           
  Partials      643      643           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 87131a0...7351644. Read the comment docs.

Copy link
Contributor

@wild-endeavor wild-endeavor left a comment

Choose a reason for hiding this comment

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

i think it's fine but do you want to look @eapolinario ?

@yindia yindia merged commit 19d977e into master Oct 29, 2021
reverson pushed a commit to reverson/flytekit that referenced this pull request May 27, 2022
* Refactor resource limit in pytorch/tensorflow plugin

Signed-off-by: Yuvraj <[email protected]>
Signed-off-by: Robert Everson <[email protected]>
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.

4 participants