Skip to content

Commit

Permalink
Merge pull request #1261 from guardian/lp-update-banditData
Browse files Browse the repository at this point in the history
Add Roulette to banditData
  • Loading branch information
LAKSHMIRPILLAI authored Dec 13, 2024
2 parents d554f88 + 701cb69 commit 50bae30
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/server/bandit/banditData.ts
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ async function buildBanditDataForTest(test: BanditTestConfig): Promise<BanditDat
// Return config for each bandit methodology in this test
function getBanditTestConfigs<V extends Variant, T extends Test<V>>(test: T): BanditTestConfig[] {
const bandits: Methodology[] = (test.methodologies ?? []).filter(
(method) => method.name === 'EpsilonGreedyBandit',
(method) => method.name === 'EpsilonGreedyBandit' || method.name === 'Roulette',
);
return bandits.map((method) => ({
testName: method.testName ?? test.name, // if the methodology should be tracked with a different name then use that
Expand Down

0 comments on commit 50bae30

Please sign in to comment.