Skip to content

Conversation

@sayakpaul
Copy link
Member

The values in user_agent are pretty non-identifiable. But LMK if you think otherwise.

I have also made it so that the logging is configurable through an env var.

@sayakpaul sayakpaul requested a review from danieldk October 14, 2025 09:00
@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Comment on lines +533 to +536
"kernels": __version__,
"torch": torch.__version__,
"build_variant": build_variant(),
"file_type": "kernel",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks non-revealing 👍, though I would be in favor of mentioning in the docs that we are collecting this information + how to switch it off.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure! Which part of the docs should we put this in?

Copy link
Member

@danieldk danieldk Oct 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure. Maybe an item here https://huggingface.co/docs/kernels/faq with an additional question like How can I disable kernel reporting in the user-agent?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

Co-authored-by: Daniël de Kok <me@danieldk.eu>
@danieldk danieldk merged commit b182cd3 into main Oct 16, 2025
13 checks passed
@danieldk danieldk deleted the log-telemetry branch October 16, 2025 18:16
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.

3 participants