hnl1

Results 7 comments of hnl1

同问

这不是强推不强推的事儿,这个是对 push 的限制,加上 protect 就不能 push,去掉就能 push,和 force 与否没有一毛钱关系

> 这不是强推不强推的事儿,这个是对 push 的限制,加上 protect 就不能 push,去掉就能 push,和 force 与否没有一毛钱关系 不是,错了,@LeachZhou 说的是对的。刚刚我实验了。 只要一个分支加到 protect 中,就不能再 force push 了。此时能不能 push 是由前面的选项来决定的,选择了你的身份能 push,那就能 push;选择了能 merge,那就能 merge。

I found another problem at https://github.com/huggingface/lerobot/blob/638d411cd3acf32c28d8c2120f3c41bda8bb15d4/examples/2_evaluate_pretrained_policy.py#L7C1-L7C27 There is an extra backtick (`) at the end, which causes the command execution to fail.

@tc-huang You're right. I just checked PR #631 , and the changes indeed solved the problem of [1_load_lerobot_dataset.py L132](https://github.com/huggingface/lerobot/blob/638d411cd3acf32c28d8c2120f3c41bda8bb15d4/examples/1_load_lerobot_dataset.py#L132)

I found another problem at https://github.com/huggingface/lerobot/blob/638d411cd3acf32c28d8c2120f3c41bda8bb15d4/examples/2_evaluate_pretrained_policy.py#L73 The original image already has the channels at the end. The operation here is to move the channels to the front. The annotation is...

I will submit a PR to fix the other issues I've found, and then close this issue.