Deskana has uploaded a new change for review.
https://gerrit.wikimedia.org/r/265784
Change subject: Improve documentation of getEventLogSamplingID method in
WikipediaApp
......................................................................
Improve documentation of getEventLogSamplingID method in WikipediaApp
This patch improves the documentation of the getEventLogSamplingID method in
WikipediaApp, to make it clearer what it can and cannot be used for. This
should remove some ambiguity for any new contributors to the code base.
Change-Id: I03ba747cb123a6881b148c565bfbb487dd867662
---
M app/src/main/java/org/wikipedia/WikipediaApp.java
1 file changed, 8 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/apps/android/wikipedia
refs/changes/84/265784/1
diff --git a/app/src/main/java/org/wikipedia/WikipediaApp.java
b/app/src/main/java/org/wikipedia/WikipediaApp.java
index 6373d35..c0bd535 100644
--- a/app/src/main/java/org/wikipedia/WikipediaApp.java
+++ b/app/src/main/java/org/wikipedia/WikipediaApp.java
@@ -378,8 +378,14 @@
}
/**
- * Get an integer-valued random ID for event log sampling. This value will
persist for the
- * lifetime of the app.
+ * Get an integer-valued random ID. This is typically used to determine
global EventLogging
+ * sampling, that is, whether the user's instance of the app sends any
events or not. This is a
+ * pure technical measure which is necessary to prevent overloading
EventLogging with too many
+ * events. This value will persist for the lifetime of the app.
+ *
+ * Don't use this method when running any kind of tests (such as A/B
tests) to determine whether
+ * or not the user falls into a control or test group, as that would
introduce sampling biases
+ * which would invalidate the test.
* @return Integer ID for event log sampling.
*/
@IntRange(from = 0)
--
To view, visit https://gerrit.wikimedia.org/r/265784
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I03ba747cb123a6881b148c565bfbb487dd867662
Gerrit-PatchSet: 1
Gerrit-Project: apps/android/wikipedia
Gerrit-Branch: master
Gerrit-Owner: Deskana <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
