Repository navigation
fix(charts): replace deprecated watermarks with a source suffix / 将已弃用水印改为来源后缀 - #1509
Merged
Merged
Conversation
Revert b39ac9c and mark deprecated models in the existing source caption instead. Preserve PNG exports, unofficial attribution, and Chinese localization. 中文:移除已弃用模型的背景水印,改为在来源署名后添加状态后缀;保留 PNG 导出、非官方来源说明和中文文案。
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Bugbot is paused — your team has an unpaid invoiceBugbot is paused because your team has an unpaid invoice. Pay it from the billing page in the Cursor dashboard and Bugbot resumes automatically. (requestId: 50136433-4d78-4eba-94d7-829150b4d27a) |
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reverts b39ac9c (#1503), removing the large deprecated-model background watermarks. Deprecated inference charts instead show
Source: SemiAnalysis InferenceX™ (deprecated)in the existing caption.The suffix follows the model's existing category and leaves active models unchanged. The shared caption covers scatter, GPU, and power-timeline charts, including PNG exports and unofficial-run attribution. Chinese captions use
(已弃用). Existing logo and unofficial watermarks are unchanged.This is presentation-only. Benchmark data, model categories, and view API filters are unchanged; the API documentation and packaged skill now describe the source suffix.
Validation
AI model disclosure
GPT-6 Astra prepared the code, tests, and PR text.
中文
回退 b39ac9c(#1503),移除已弃用模型的大号背景水印,改为在现有来源署名后添加
(已弃用),英文页面显示(deprecated)。后缀沿用模型现有的分类,不影响仍在维护的模型。散点图、GPU 图和功耗时间线共用该来源说明,PNG 导出和非官方 run overlay 也保留后缀;原有 logo 和非官方水印不变。
此变更仅影响展示,不改变 benchmark 数据、模型分类或视图 API 的筛选行为。API 文档与打包 skill 已同步更新。
类型检查、lint、格式、排版和 skill 完整性检查通过。新增浏览器测试覆盖六个已弃用模型、一个仍在维护的模型、PNG 导出、非官方 overlay 和中文移动端布局;CI 单元测试、所有 Chrome/Firefox E2E 分片及 Vercel 预览构建通过。
本地单元测试有两个五秒超时,使用默认超时单独重跑后均通过。本地 Electron smoke 首次通过 98/99 项,中文 404 页面触发 Next.js 开发模式的 performance 时间戳异常;完整重跑该 spec 后通过 24/24 项。