Skip to content

Conversation

@heiruwu
Copy link
Contributor

@heiruwu heiruwu commented Jul 18, 2024

Because

  • We are going to remove data serialization between model-backend and ray-server to improve the DX for implementing a custom model
  • To support local model image self-test, ray application must be able to accept payload in JSON object for ease of use

This commit

  • adopt latest contract for ray application
  • redesign and rewrite IO help functions

@heiruwu heiruwu requested a review from joremysh as a code owner July 18, 2024 16:14
@linear
Copy link

linear bot commented Jul 18, 2024

@heiruwu heiruwu merged commit bc14981 into main Jul 18, 2024
@heiruwu heiruwu deleted the heiru/INS-5271 branch July 18, 2024 16:17
heiruwu added a commit that referenced this pull request Aug 7, 2024
Because

- We are going to remove data serialization between `model-backend` and
`ray-server` to improve the DX for implementing a custom model
- To support local model image self-test, ray application must be able
to accept payload in JSON object for ease of use

This commit

- adopt latest contract for ray application
- redesign and rewrite IO help functions
heiruwu added a commit that referenced this pull request Sep 3, 2024
Because

- We are going to remove data serialization between `model-backend` and
`ray-server` to improve the DX for implementing a custom model
- To support local model image self-test, ray application must be able
to accept payload in JSON object for ease of use

This commit

- adopt latest contract for ray application
- redesign and rewrite IO help functions
heiruwu added a commit that referenced this pull request Sep 5, 2024
Because

- We are going to remove data serialization between `model-backend` and
`ray-server` to improve the DX for implementing a custom model
- To support local model image self-test, ray application must be able
to accept payload in JSON object for ease of use

This commit

- adopt latest contract for ray application
- redesign and rewrite IO help functions
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