Comfy-Org / comfy-cli

Command Line Interface for Managing ComfyUI
https://docs.comfy.org/comfy-cli/getting-started
GNU General Public License v3.0
260 stars 41 forks source link

Add docs for --no-enable-telemetry #128

Open christian-byrne opened 3 months ago

christian-byrne commented 3 months ago

Describe the solution you'd like The --no-enable-telemetry option should be in the help screen or documentation.

Additional context I spent a long time attempting to bypass that prompt while trying to use comfy-cli in a sandbox 💀. Also when I run this:

python3 -m venv venv; source venv/bin/activate && \
firejail --noprofile --whitelist="/home/c_byrne/temp/venv/*" -- bash -c "python -m pip install comfy-cli; comfy-cli tracking disable; comfy-cli --workspace ./mock-ComfyUI"

it will say:

Tracking is now disabled.
Do you agree to enable tracking to improve the application? [y/N]: 
robinjhuang commented 3 months ago

So you can do this to bypass telemetry prompt.

comfy --skip-prompt --no-enable-telemetry env

And add --skip-prompt to subsequent commands to skip those. Did you want this in the README?

christian-byrne commented 3 months ago

I will use that, thanks for the info.

For for README it could be good. Like in a section on using it for automation or cloud setup. It's going to be the CLI for node package management system right? So I thought, it will be common to use it to setup environments automatically, but I couldn't figure out how to bypass the prompt. Sorry if posting this issue caused bad things to happen.