Skip to content

Conversation

@mohamad-tohidi
Copy link

#642
i added a simple encoder, for triton inference server

i also added a semantic-router[triton] option for installing, for easier integration

@jamescalam jamescalam changed the base branch from main to feat/triton-encoder November 4, 2025 13:31
@jamescalam
Copy link
Member

hey @mohamad-tohidi code-wise looks good, could you just run uv sync locally to update your uv.lock file (and commit), resolve any linting issues flagged by the CI — and if all is good from there I'll run our integration tests against the PR and add a triton integration doc before merging. Thanks!

@mohamad-tohidi
Copy link
Author

@jamescalam thank you for the response
should i create a new pull request? there was minor also miner bugs that i fixed

@jamescalam
Copy link
Member

you can keep it going in this PR - upto you

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