New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sending anonymous analytics #1204
Conversation
@@ -0,0 +1,30 @@ | |||
"""Module for anonymous analytics tracking.""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add here another sentence clarifying the same as inside the documentation...
(no personal data etc. will ever be sent. only a single event upon import)
@@ -90,3 +90,17 @@ Start Working with the Package | |||
Now it's time to :doc:`check out </index>` deepchecks! | |||
|
|||
|
|||
Anonymous Usage Metrics |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the name telemetry is more common in this cases
deepchecks/__init__.py
Outdated
try: | ||
disable = os.environ.get('DEEPCHECKS_DISABLE_LATEST', 'false').lower() == 'true' | ||
disable = os.environ.get('DISABLE_DEEPCHECKS_ANONYMOUS_TRACKING', False) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is already if disabled inside the function
disable = os.environ.get('DISABLE_DEEPCHECKS_ANONYMOUS_TRACKING', False) | |
send_anonymous_import_event() |
What does this implement/fix? Explain your changes.
Any other comments?
Thanks for contributing a pull request! Please ensure you have taken a look at
the contribution guidelines: https://github.com/deepchecks/deepchecks/blob/main/CONTRIBUTING.rst