Steven Palma

I'm closing this PR as it has been superseded by https://github.com/huggingface/lerobot/pull/2100

Using a synchronized queue is indeed another valid implementation 👍🏻 IMO: Locks: ✅ Low overhead ❌ Manual synchronization Queues: ✅ Auto synchronization & event-driven ❌ Higher overhead, memory complexity increases...

Is this ready for merging?

Hello @jadechoghari, thanks for your contribution! This LGTM 😄

Hello @pablovela5620 , @theophilegervet and @IrvingF7, Thanks for spotting this out! I'm currently working on a fix, it requires some refactoring but you can expect a PR in the following...

Sorry forgot to update the status of this issue, it was solved some weeks ago here: https://github.com/huggingface/lerobot/pull/812

Hello @villekuosmanen , thanks for your contribution! Is this ready for a review? Can you please add the PR description information? Thanks!

Calibration process has changed significantly in the previous weeks, can you confirm you still have an issue with this?

PI0Fast has been deprecated in https://github.com/huggingface/lerobot/pull/2203 Please prioritize the use of the other PI0 family models

Hello @JayceWen ! Thanks for your contribution. Indeed, someone from the team needs to review this PR before it can get merged to upstream. I will ping the policy experts...