Skip to content

Conversation

@heiruwu
Copy link
Contributor

@heiruwu heiruwu commented Mar 1, 2024

Because

  • model-backend needs Ray CLI to deploy dockerized application

This commit

  • return deployment handle for CLI to reference

@linear
Copy link

linear bot commented Mar 1, 2024

@codecov
Copy link

codecov bot commented Mar 1, 2024

Codecov Report

Attention: Patch coverage is 0% with 12 lines in your changes are missing coverage. Please review.

Project coverage is 24.48%. Comparing base (cc61b85) to head (bfe7c9a).

Files Patch % Lines
instill/helpers/ray_config.py 0.00% 12 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #107      +/-   ##
==========================================
- Coverage   24.49%   24.48%   -0.02%     
==========================================
  Files         189      189              
  Lines        6609     6613       +4     
  Branches      782      782              
==========================================
  Hits         1619     1619              
- Misses       4973     4977       +4     
  Partials       17       17              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@heiruwu heiruwu merged commit 5423b4d into main Mar 1, 2024
@heiruwu heiruwu deleted the INS-3794 branch March 1, 2024 11:48
heiruwu added a commit that referenced this pull request Mar 13, 2024
Because

- `model-backend` needs `Ray` CLI to deploy dockerized application

This commit

- return deployment handle for CLI to reference
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: 👋 Done

Development

Successfully merging this pull request may close these issues.

3 participants