Results 17 comments of NP

I had a similar issue. It's likely that you have a redis-server instance running on the same address that you're trying to use. So if you're using Linux (Ubuntu) try:...

I think it would be a good idea to add direct tests for this part of the code-base. Currently, we only cover it indirectly in the [MPC tensor tests](https://github.com/OpenMined/SyMPC/blob/main/tests/sympc/tensor/mpc_tensor_test.py). This...

@madhavajay @gmuraru - in terms of integration, what do you think would be best: to start adding part of the code-base bit by bit (different PRs) or to have one...

> Here, it seems that we will have SyMPC/src/sympc/ for the Python code, and SyMPC/src/src/ for the Rust code. Indeed, I left it like that because it's the same structure...

@tholop - I've moved the Rust tests to a separate folder. Depending on what the final structure, this folder might stay there or be at the root of the project,...

> Could we also have a basic function added and called in the Python code? > I am thinking at the `ispointer` function. Yes - I am thinking of adding...

> @arturomf94 That can be another interesting story if you're up for the challenge :) yeah! for sure! please assign this to me :)

Hey! FWIW - I manually added the extra tokens and used the [`set_tokenizer`](https://github.com/vllm-project/vllm/blob/746709642c81aa22926765aef67e086a15aef076/vllm/entrypoints/llm.py#L175) method to update the tokenizer. The output changed, but it seems that there's still something wrong with...