Skip to content

Disable telemetry when DISABLE_TELEMETRY is set - #1686

Merged
anton-l merged 1 commit into
huggingface:mainfrom
w4ffl35:main
Dec 13, 2022
Merged

Disable telemetry when DISABLE_TELEMETRY is set#1686
anton-l merged 1 commit into
huggingface:mainfrom
w4ffl35:main

Conversation

@w4ffl35

@w4ffl35 w4ffl35 commented Dec 13, 2022

Copy link
Copy Markdown
Contributor

fixed #1685 - disables telemetry when DISABLE_TELEMETRY and HF_HUB_OFFLINE is set

fixed huggingface#1685 - disables telemetry when DISABLE_TELEMETRY and HF_HUB_OFFLINE is set
@HuggingFaceDocBuilderDev

HuggingFaceDocBuilderDev commented Dec 13, 2022

Copy link
Copy Markdown

The documentation is not available anymore as the PR was closed or merged.

@anton-l

anton-l commented Dec 13, 2022

Copy link
Copy Markdown
Member

Thank you @w4ffl35!
Thankfully the UA formatter caught the error and didn't send any additional telemetry other than the python and diffusers versions

ua = f"diffusers/{__version__}; python/{sys.version.split()[0]}; session_id/{SESSION_ID}"
if DISABLE_TELEMETRY:
return ua + "; telemetry/off"

@anton-l
anton-l merged commit 07f9550 into huggingface:main Dec 13, 2022
sliard pushed a commit to sliard/diffusers that referenced this pull request Dec 21, 2022
fixed huggingface#1685 - disables telemetry when DISABLE_TELEMETRY and HF_HUB_OFFLINE is set
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request Dec 25, 2023
fixed huggingface#1685 - disables telemetry when DISABLE_TELEMETRY and HF_HUB_OFFLINE is set
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

It is not possible to disable telemetry

3 participants