Small update in installation instructions#778
Conversation
|
Looks good! Should we also add the one-time usage option using |
ataffanel
left a comment
ataffanel
left a comment
There was a problem hiding this comment.
I think the Venv situation needs to be a bit more documented.
docs/installation/install.md
Outdated
|
|
||
| ```bash | ||
| pip install cfclient | ||
| pip3 install cfclient |
There was a problem hiding this comment.
This, and the UV version, requires a VENV to be created on all modern Python environment, this should also be documented I guess (and the start of the file mofidied to change venv from recomended to mandatory).
docs/installation/install.md
Outdated
| Choose the installation method that fits your use case: | ||
|
|
||
| If you plan to use the client to control the Crazyflie, we highly recommend installing the latest release using pip, as this is well tested and stable: | ||
| - **For regular use**: Install from PyPI using either `pip` or `uv` (both work equally well) |
There was a problem hiding this comment.
Should we advise uv since it does work much better than raw pip. It does need to be installed though.
docs/installation/install.md
Outdated
| Choose the installation method that fits your use case: | ||
|
|
||
| If you plan to use the client to control the Crazyflie, we highly recommend installing the latest release using pip, as this is well tested and stable: | ||
| - **For regular use**: Install from PyPI using either `pip` or `uv` (both work equally well) |
There was a problem hiding this comment.
For regular use, maybe documenting uvx cfclient is a good option since it does not require to setup a separate venv just to run the client?
ataffanel
left a comment
ataffanel
left a comment
There was a problem hiding this comment.
I have a last nit about the description of UVX, otherwise this is a great improvement.
3 participants
Added uv and updated pip. Clarified what is for regular usage or development.