Skip to content

[Fix] Link VideoGenX from the unofficial run visualizer comment / [Fix] 在非官方运行可视化评论中加入 VideoGenX 链接 - #5083

Open
edwingao28 wants to merge 1 commit into
mainfrom
fix/vgx-unofficial-run-link
Open

edwingao28 wants to merge 1 commit into
mainfrom
fix/vgx-unofficial-run-link

Conversation

@edwingao28

Copy link
Copy Markdown
Collaborator

Summary

The unofficial run visualizer comment links only /inference and /evaluation. Video sweep results are shown on the VideoGenX page, so a video run's comment had no link that displayed its results. When the selected matrix has video rows, the comment now adds View unofficial run (video): https://inferencex.semianalysis.com/vgx?unofficialRun=<run>. Sweeps without video rows get the same comment as before.

The app side, where /vgx accepts ?unofficialRun= and plots frame-count workloads like Wan2.2, is in the VideoGenX unofficial run fix.

Validation

  • The workflow YAML parses. HAS_VIDEO uses the same single_node.video checks as validate-video-results.
  • Not verified: an actual comment post. That happens on the next labelled video sweep.
中文

非官方运行可视化评论只给出 /inference 和 /evaluation 链接,而视频 sweep 结果显示在 VideoGenX 页面,因此视频运行的评论里没有能显示其结果的链接。现在当所选矩阵包含视频配置时,评论会额外加入 /vgx?unofficialRun=<run> 链接。不含视频配置的 sweep 评论保持不变。应用侧的修复见 VideoGenX 非官方运行修复。

未验证:实际发帖效果,需等下一次带标签的视频 sweep。

AI disclosure: prepared with Perplexity Computer (orchestrator model Claude Opus 5.5).

Created with Perplexity

…官方运行可视化评论中加入 VideoGenX 链接

Sweeps with video rows now also link /vgx?unofficialRun=<run>, where video
results are shown; the inference and evaluation links are unchanged.

中文:包含视频配置的 sweep 在评论中额外给出 /vgx?unofficialRun=<run> 链接,
视频结果在该页面显示;推理与评测链接不变。

Co-authored-by: Perplexity <noreply@perplexity.ai>

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review found no issues

No high-confidence issues detected in this change.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant