Skip to content

Conversation

quic-sanirudh
Copy link
Contributor

This patch modifies the get_executor_from_factory for relax to support accepting a string that points to an already exported shared library. This allows us to run models that were already compiled through the RPC executor.

@cbalint13 cbalint13 self-assigned this Jul 16, 2024
Copy link
Contributor

@cbalint13 cbalint13 left a comment

Choose a reason for hiding this comment

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

It looks good to me.

This patch modifies the `get_executor_from_factory` for relax to support
accepting a string that points to an already exported shared library.
This allows us to run models that were already compiled through the RPC
executor.
@quic-sanirudh quic-sanirudh force-pushed the support_running_existing_so_through_rpc branch from 9e74749 to 1b89c86 Compare July 16, 2024 11:01
@cbalint13 cbalint13 merged commit 51d7c5e into apache:main Jul 16, 2024
@quic-sanirudh quic-sanirudh deleted the support_running_existing_so_through_rpc branch July 17, 2024 05:03
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.

2 participants