Skip to content

Commit

Permalink
fix redundant Math.min()
Browse files Browse the repository at this point in the history
  • Loading branch information
jurajmajerik committed Apr 16, 2024
1 parent d66c19c commit af439b2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions frontend/src/scenes/experiments/ExperimentView/components.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -417,7 +417,7 @@ export function ActionBanner(): JSX.Element {
// Win probability only slightly over 0.9 and the recommended sample/time just met -> proceed with caution
if (
experimentInsightType === InsightType.FUNNELS &&
funnelResultsPersonsTotal > Math.min(recommendedSampleSize) + 50 &&
funnelResultsPersonsTotal > recommendedSampleSize + 50 &&
winProbability < 0.93
) {
return (
Expand All @@ -431,7 +431,7 @@ export function ActionBanner(): JSX.Element {

if (
experimentInsightType === InsightType.TRENDS &&
actualRunningTime > Math.min(recommendedRunningTime) + 2 &&
actualRunningTime > recommendedRunningTime + 2 &&
winProbability < 0.93
) {
return (
Expand Down

0 comments on commit af439b2

Please sign in to comment.