Skip to content

knowledge(vllm-omni): 1 change(s) from intake - #341

Merged
tzhouam merged 1 commit into
mainfrom
kb/vllm-omni/vllm-omni-intake-0e28babd348d
Oct 9, 2026
Merged

tzhouam merged 1 commit into
mainfrom
kb/vllm-omni/vllm-omni-intake-0e28babd348d

Conversation

@tzhouam

@tzhouam tzhouam commented Oct 9, 2026

Copy link
Copy Markdown
Collaborator

Generated automatically by the Copilot knowledge service (changeset vllm-omni-intake-0e28babd348d).

op page rule
add repos/vllm-omni/models/personaplex/rules.md VLLM-OMNI-PR8670-AMD-PPLEX-NIGHTLY

Generator: zcode:GLM-5.3 · judge: codex:gpt-6-sol:medium.
The publisher merges it only if its local gate passes on the exact merge result.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-09T17:59:12.846761Z 3484e01 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@tzhouam
tzhouam merged commit 982a865 into main Oct 9, 2026
3 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3484e019e3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

时仍只有一次embedding调用。Stage0真实embedding integration证明first append只一次prefill,
second append只含live frame;组合stack的E2E结果不能单独归因此PR。^[PR #7481]

## VLLM-OMNI-PR8670-AMD-PPLEX-NIGHTLY — PersonaPlex temporal 压力用例只在 AMD nightly 串行承接

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Give the nightly rule an auditable PPLEX ID

When this owner page is supplied to knowledge/tools/check_review_report.py, RULE_DEFINITION_RE at lines 62–66 recognizes only IDs shaped like PREFIX-<digits><optional letter>, matching the page's declaration that only PPLEX-数字字母 IDs are auditable. This PR-specific heading is therefore omitted by extract_rule_ids, so the mandatory owner-rule audit can pass without reviewing this new CI contract. Assign it a valid PPLEX-* ID and include that ID in the Direct quick-entry map.

Useful? React with 👍 / 👎.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant