Skip to content

Commit 6d3dc1f

Browse files
authored
[DISCO-4025] fix: fetch engagement metric default (#1360)
1 parent 6eef2e4 commit 6d3dc1f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

merino/providers/suggest/adm/provider.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ def _fetch_engagement_metrics(self, suggestion: PyAmpResult) -> EngagementMetric
214214
engaged, attempted = 1, 1
215215
if self.engagement_data and (metrics := self.engagement_data.amp.get(advertiser)):
216216
attempted = int(metrics.get("impressions", attempted))
217-
engaged = int(metrics.get("clicks", attempted))
217+
engaged = int(metrics.get("clicks", engaged))
218218
return EngagementMetrics(engaged=engaged, attempted=attempted)
219219

220220
def _is_thompson_eligible(self, client_variants: list[str]) -> bool:

0 commit comments

Comments
 (0)