Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@wanglining97
Copy link
Collaborator

@wanglining97 wanglining97 commented Nov 3, 2025

施工中

@wanglining97 wanglining97 force-pushed the develop/embedding_grpc_server branch from db3214a to 223cca9 Compare November 3, 2025 14:10

grpc::Status health(grpc::ServerContext* context, const EmbeddingHealthRequestPB* request, EmptyPB* writer);

private:
Copy link
Collaborator

Choose a reason for hiding this comment

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

engine 里面的其他接口打算实现吗?worker_status? cache_status

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

意思应该是local rpc server里实现worker_status cache_status,替代ops吧,是要的

py::object custom_module = model.attr("custom_module");
py::object py_render = model.attr("custom_module").attr("renderer");
py::object py_tokenizer = model.attr("tokenizer");
py::object py_handler = model.attr("custom_module").attr("handler");
Copy link
Collaborator

Choose a reason for hiding this comment

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

后处理还没处理

return await self.embedding(request, raw_request)

def _handle_exception(self, request: Dict[str, Any], e: BaseException):
exception_json = format_exception(e)
Copy link
Collaborator

Choose a reason for hiding this comment

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

还需要吗

Copy link
Collaborator Author

@wanglining97 wanglining97 Nov 4, 2025

Choose a reason for hiding this comment

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

_handle_exception这种函数等后面集中处理一下

@wanglining97 wanglining97 force-pushed the develop/embedding_grpc_server branch from 57743c4 to 6811469 Compare November 7, 2025 03:15
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.

3 participants