Skip to content

Commit f501dbb

Browse files
committed
fix(compare): dedup keywords so same-vendor pairs don't repeat vendor name
H100 vs H200 was producing 'NVIDIA, NVIDIA' in the Dataset keywords list, and MI300X vs MI325X was producing 'AMD, AMD'. Wrap the keyword list in a Set so each value appears at most once. Cross-vendor pairs still get both 'NVIDIA, AMD'.
1 parent ac9b5d7 commit f501dbb

1 file changed

Lines changed: 14 additions & 12 deletions

File tree

packages/app/src/lib/compare-ssr.ts

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -912,18 +912,20 @@ export function buildJsonLd(
912912
measurementTechnique:
913913
'Open-source automated GPU CI/CD inference benchmark (github.com/SemiAnalysisAI/InferenceX)',
914914
keywords: [
915-
'AI inference benchmark',
916-
'GPU comparison',
917-
variant === 'per-dollar' ? 'cost per million tokens' : 'inference latency',
918-
variant === 'per-dollar' ? 'performance per dollar' : 'tokens per second',
919-
model.label,
920-
aLabel,
921-
bLabel,
922-
HW_REGISTRY[a]?.vendor,
923-
HW_REGISTRY[b]?.vendor,
924-
]
925-
.filter(Boolean)
926-
.join(', '),
915+
...new Set(
916+
[
917+
'AI inference benchmark',
918+
'GPU comparison',
919+
variant === 'per-dollar' ? 'cost per million tokens' : 'inference latency',
920+
variant === 'per-dollar' ? 'performance per dollar' : 'tokens per second',
921+
model.label,
922+
aLabel,
923+
bLabel,
924+
HW_REGISTRY[a]?.vendor,
925+
HW_REGISTRY[b]?.vendor,
926+
].filter(Boolean),
927+
),
928+
].join(', '),
927929
...(datePublished && { datePublished }),
928930
...(dateModified && { dateModified }),
929931
creator: {

0 commit comments

Comments
 (0)