Add support for Windows

[brainwavecoder9]

What this does

Windows uses COM ports not what is expected for Linux/Mac. This commit adds support for Windows OS. It does this by:

• checking to see if OS is Windows, or other
• if Windows, use a new process
• if other, use existing process

There is a dependency (pyserial) but it is already in the environment so no changes there. Once the port is identified it can be added to the .yaml the same as it is today. Although it will be referenced differently (e.g., COM7 vs. /dev/ttyACM) the rest of the functionality still works.

How it was tested

Successful in Windows environment, and theoretically won't affect others, but does need to be confirmed before merge.

How to checkout & try? (for the reviewer)

No change to existing process. To use, simply execute the existing script, e.g., python lerobot/scripts/find_motors_bus_port.py

[Cadene]

Looks good to me! Thanks for the improvements to print + comments ;)
Running tests and merging.

[Cadene]

Style check failing. Run our pre-commit ;)

pre-commit run --all-files

https://github.com/huggingface/lerobot/blob/main/CONTRIBUTING.md#submitting-a-pull-request-pr

[brainwavecoder9]

Oh you mean I'm actually supposed to follow the instructions for contributing 😅

I thought "no way this won't pass" but here we are... updates made!

[brainwavecoder9]

OK @Cadene this one isn't on me my test passed 😄

System.IO.IOException: No space left on device : '/home/runner/runners/2.320.0/_diag/Worker_20241101-143253-utc.log'
Unhandled exception. System.IO.IOException: No space left on device : '/home/runner/runners/2.320.0/_diag/Worker_20241101-143253-utc.log'

If this is something you want me to have a look at let me know, I've done lots of infra.

[Cadene]

Thanks!

[Cadene]

@brainwavecoder9 If you find any other compatibility issue, please let me know. I will be more reactive this time ;)