feat(compare-per-dollar): embed SemiAnalysis logo in chart PNG footer#403
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@claude resolve conflicts. Move the watermark on top of the cost comparison chart with low opacity |
|
Claude finished @Oseltamivir's task in 8m 6s —— View job Done
What changed vs. the previous PR head
Preview the PNG (Vercel preview): Not done
|
Resolve conflict in compare-per-dollar PNG route by taking master's trackServer try/catch refactor and re-applying the SemiAnalysis logo as a centered low-opacity watermark on top of the cost-comparison chart rather than in the footer. Co-authored-by: Bryan Shan <Oseltamivir@users.noreply.github.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 505324b. Configure here.
|
@claude increase the size by 3x |
|
Claude finished @Oseltamivir's task in 1m 53s —— View job Done
What changedWatermark scaled by 3x in
The half-width offset stays in step with the height because the logo's intrinsic aspect ratio (1920×825) is preserved when only Opacity ( Preview: open the Vercel preview PNG at |
Co-authored-by: Bryan Shan <Oseltamivir@users.noreply.github.com>

Summary
getLogoSrc()helper (1:1 withopengraph-image.tsx) that readspublic/brand/logo-color.pnginto a base64 data URL.Promise.all.28 * R= 56 px native) above the existinginferencex.semianalysis.comtext in the bottom-right footer of the indexed chart PNG.Motivation: the
/compare-per-dollar/*chart PNGs are linked from the image sitemap and intended for Google Images / republication. Adding the SemiAnalysis wordmark strengthens attribution when the image is surfaced outside the site.Test plan
/compare-per-dollar/deepseek-v4-gb200-vs-gb300/performance-per-dollar.pngand confirm the logo renders aboveinferencex.semianalysis.comin the bottom-right corner, sharp at 2x.deepseek-r1-b200-vs-gb300,glm-5-1-gb300-vs-mi355x) to confirm consistent placement across pair variations.<img>on the page (packages/app/src/app/compare-per-dollar/[slug]/page.tsx) still loads — the PNG URL is unchanged.Note
Low Risk
Visual-only change to OG PNG rendering with graceful logo load failure; no API, auth, or data logic changes.
Overview
Adds a cached
getLogoSrc()helper (same pattern asopengraph-image.tsx) that readspublic/brand/logo-color.pnginto a base64 data URL, with failures resolving tonullso PNG generation still succeeds.Benchmark loading now runs in
Promise.allalongside the logo read. When the logo is available, the compare chart renders a centered, low-opacity (12%) watermark over the plot area via an absolutely positioned<img>—intended to strengthen SemiAnalysis attribution when/compare-per-dollar/*/performance-per-dollar.pngis reused off-site (e.g. image search).Reviewed by Cursor Bugbot for commit 44ca1ad. Bugbot is set up for automated code reviews on this repo. Configure here.