Skip to content

Commit

Permalink
feat(experiments): Add primary and secondary metric explanation (Post…
Browse files Browse the repository at this point in the history
  • Loading branch information
danielbachhuber authored Feb 3, 2025
1 parent 5aa6180 commit b4cf86b
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions frontend/src/scenes/experiments/MetricsView/MetricsView.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -259,8 +259,15 @@ export function MetricsView({ isSecondary }: { isSecondary?: boolean }): JSX.Ele
<div className="border rounded bg-bg-light pt-6 pb-8 text-muted mt-2">
<div className="flex flex-col items-center mx-auto space-y-3">
<IconAreaChart fontSize="30" />
<div className="text-sm text-center text-balance">
Add up to {MAX_PRIMARY_METRICS} {isSecondary ? 'secondary' : 'primary'} metrics.
<div className="text-sm text-center text-balance max-w-sm">
<p>
Add up to {MAX_PRIMARY_METRICS} {isSecondary ? 'secondary' : 'primary'} metrics.
</p>
<p>
{isSecondary
? 'Secondary metrics provide additional context and help detect unintended side effects.'
: 'Primary metrics represent the main goal of the experiment and directly measure if your hypothesis was successful.'}
</p>
</div>
{isSecondary ? <AddSecondaryMetric /> : <AddPrimaryMetric />}
</div>
Expand Down

0 comments on commit b4cf86b

Please sign in to comment.