-
Notifications
You must be signed in to change notification settings - Fork 18
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
Is telemetry really needed? #114
Comments
Hi @maximlt, this is a question I have asked myself quite a bit. These are the high-level reasons I recently made this change:
I'm sorry you feel this is a red flag. IMO it reflects a paradigm shift in (a) how people make product decisions in open source project, and (b) the increasingly commercial aspect of open source. For context: I am working on more powerful adjacent offerings to nbmake. This is financially risky but I do it because I like to serve developers like you -- telemetry helps bound the risk. From your perspective, does this make you feel any more comfortable with the decision? Is there are better middle ground I could strike here? E: Disabled telemetry by default. My main goal with this was to verify that this was actually being used (not just downloaded). I determined it was being invoked ~4K every weekday. |
removed the disabled metrics code in #115, closing this, thanks for the feedback |
Hi @alex-treebeard,
I've been looking for a Pytest plugin to run notebooks and found your project to be an interesting alternative to e.g
nbval
. Looking at the repo I found out that you recently added some telemetry in #113. Even if it seems it's possible to deactivate it, I found that very surprising for an open-source project of this dimension. I usually see bigger projects implementing telemetry, they're usually backed by some start-up/company and need data to report progress to their investors. I don't know if this is the case for nbmake. To me at least, this is definitely a red flag and discards nbmake as a tool I would add to my testing framework.The text was updated successfully, but these errors were encountered: