Skip to content

Conversation

aivanou
Copy link
Contributor

@aivanou aivanou commented Nov 16, 2021

Summary: Remove ray scheduler from registry for now since it is not ready

Differential Revision: D32473942

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Nov 16, 2021
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D32473942

aivanou added a commit to aivanou/torchx-1 that referenced this pull request Nov 16, 2021
Summary:
Pull Request resolved: pytorch#350

Remove ray scheduler from registry for now since it is not ready

Differential Revision: D32473942

fbshipit-source-id: 308d5b094b70bfe1a9f5e93a7037251ad688e9b9
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D32473942

aivanou added a commit to aivanou/torchx-1 that referenced this pull request Nov 16, 2021
Summary:
Pull Request resolved: pytorch#350

Remove ray scheduler from registry for now since it is not ready

Differential Revision: D32473942

fbshipit-source-id: 9ae8748f0eed87b355d1a8ffe3546dad54ed5922
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D32473942

@codecov
Copy link

codecov bot commented Nov 16, 2021

Codecov Report

Merging #350 (73aa905) into main (e904178) will decrease coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #350      +/-   ##
==========================================
- Coverage   94.62%   94.55%   -0.08%     
==========================================
  Files          59       59              
  Lines        3087     3084       -3     
==========================================
- Hits         2921     2916       -5     
- Misses        166      168       +2     
Impacted Files Coverage Δ
torchx/schedulers/__init__.py 95.00% <ø> (-0.66%) ⬇️
torchx/schedulers/ray_scheduler.py 90.62% <0.00%> (-2.09%) ⬇️

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 e904178...73aa905. Read the comment docs.

aivanou added a commit to aivanou/torchx-1 that referenced this pull request Nov 17, 2021
Summary:
Pull Request resolved: pytorch#350

Remove ray scheduler from registry for now since it is not ready

Differential Revision: D32473942

fbshipit-source-id: 82d5f20ae86a0792e82ae1029b6c2c9c793d6758
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D32473942

aivanou added a commit to aivanou/torchx-1 that referenced this pull request Nov 17, 2021
Summary:
Pull Request resolved: pytorch#350

Remove ray scheduler from registry for now since it is not ready

Reviewed By: d4l3k

Differential Revision: D32473942

fbshipit-source-id: 1ac081456e9eea26f51b1ee05be33d9067174dbc
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D32473942

aivanou added a commit to aivanou/torchx-1 that referenced this pull request Nov 17, 2021
Summary:
Pull Request resolved: pytorch#350

Remove ray scheduler from registry for now since it is not ready

Reviewed By: d4l3k

Differential Revision: D32473942

fbshipit-source-id: 85a51a07249958d264dfe137e6f02617d926fc58
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D32473942

Summary:
Pull Request resolved: pytorch#350

Remove ray scheduler from registry for now since it is not ready

Reviewed By: d4l3k

Differential Revision: D32473942

fbshipit-source-id: 50771faf91294de245392955e81e1a06e4e9c77a
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D32473942

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 0f81544.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants