jenkins-bot has submitted this change and it was merged.

Change subject: Turn on CirrusSearch bm25 A/B test
......................................................................


Turn on CirrusSearch bm25 A/B test

Enable the 5 buckets descibed in Ia7c1f9d.

Bug: T143586
Change-Id: I5435821c57c2f0e1298fa2b4b087a58d020b8aee
---
M modules/ext.wikimediaEvents.searchSatisfaction.js
1 file changed, 19 insertions(+), 3 deletions(-)

Approvals:
  EBernhardson: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/ext.wikimediaEvents.searchSatisfaction.js 
b/modules/ext.wikimediaEvents.searchSatisfaction.js
index 12777e2..3217a9b 100644
--- a/modules/ext.wikimediaEvents.searchSatisfaction.js
+++ b/modules/ext.wikimediaEvents.searchSatisfaction.js
@@ -100,7 +100,8 @@
                function initialize( session ) {
 
                        var sessionId = session.get( 'sessionId' ),
-                               sampleSize = 200,
+                               haveSubTest = mw.config.get( 'wgDBname' ) === 
'enwiki',
+                               sampleSize = haveSubTest ? 66 : 200,
                                /**
                                 * Determines whether the user is part of the 
population size.
                                 *
@@ -149,6 +150,16 @@
                                // have a search session id, generate one.
                                if ( !session.set( 'sessionId', randomToken() ) 
) {
                                        return;
+                               }
+
+                               if ( haveSubTest && !oneIn( 3 ) ) {
+                                       session.set( 'subTest', chooseBucket( [
+                                               'bm25:control',
+                                               'bm25:allfield',
+                                               'bm25:inclinks',
+                                               'bm25:inclinks_pv',
+                                               'bm25:inclinks_pv_rev'
+                                       ] ) );
                                }
                        }
 
@@ -569,8 +580,13 @@
        // text setup, so wrap in atMostOnce to ensure it's
        // only run once.
        initSubTest = atMostOnce( function ( session ) {
-               // jshint unused:false
-               // no sub test currently running
+               if ( session.get( 'subTest' ) ) {
+                       $( '<input>' ).attr( {
+                               type: 'hidden',
+                               name: 'cirrusUserTesting',
+                               value: session.get( 'subTest' )
+                       } ).prependTo( $( 'input[type=search]' ).closest( 
'form' ) );
+               }
        } );
 
        /**

-- 
To view, visit https://gerrit.wikimedia.org/r/306948
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5435821c57c2f0e1298fa2b4b087a58d020b8aee
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/WikimediaEvents
Gerrit-Branch: master
Gerrit-Owner: DCausse <[email protected]>
Gerrit-Reviewer: DCausse <[email protected]>
Gerrit-Reviewer: EBernhardson <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to