[til] DeepSeek-V4.1-Flash Review: Pushing KV Cache Compression Limits - #92
Closed
github-actions[bot] wants to merge 2 commits into
Closed
github-actions[bot] wants to merge 2 commits into
github-actions[bot] wants to merge 2 commits into
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
Author
There was a problem hiding this comment.
Threat detection produced a warning for this pull request output.
These changes need to be scrutinized before merge and only merged after a careful manual review.
- Detection reason:
threat_detected - Review workflow run logs: https://github.com/sameerkhansf/Sameer-Khan/actions/runs/34695013396
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
Author
|
This pull request was automatically closed because it expired on 2026-09-19T13:08:43.307Z.
|
sameerkhansf
added a commit
that referenced
this pull request
Sep 28, 2026
…llowed-files - rejected-til-topics excludes PRs closed by the gh-aw expiry job (its own "automatically closed because it expired" comment): #92 and #101 were never rejected, only unreviewed. Verified on live data: [101,92,77,...] -> [77,...]. - Slugs must match ^[a-z0-9-]+$: proxy.js negotiates markdown only for /blog/[\w-]+, so #101's dotted slug would have lost its markdown twin. The fact-check pre-step fails any non-kebab post file deterministically; the writer prompts state the rule. - allowed-files content/blog/** -> content/blog/*.mdx (gh-aw's matcher is */** only, no character classes; this blocks subdirectories and non-MDX). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
sameerkhansf
added a commit
that referenced
this pull request
Sep 28, 2026
* ci(til): auto-merge TIL posts behind a required fact-check gate TIL PRs passed CI and then expired unreviewed (#92, #101), and the one-pending-post gate turned a week of scout runs into noops. Publish autonomously, using only vendor-native gh-aw controls: - create-pull-request auto-merge: squash, expires 2d (daily-til, til-writer) - threat-detection continue-on-error: false; the default let a flagged detection (#92, prompt_injection=true) still open the PR - til-fact-check: new gh-aw workflow (governance pattern) that verifies every claim against its primary source and posts a `fact-check` check run - skip-if-match: blocked scout runs no longer spend model calls - arxiv.org allowlisted; til-writer can now reach its research sources - stale "human merge" prompt text removed Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * ci(til): prefetch cited sources for fact-check; recheck on label; pin verdict to analyzed SHA Run 36388743650 hit invocation_cap_exceeded (250/250) re-running curl|grep on one README. DeterministicOps: steps: downloads every cited URL (plus HF raw README/config.json) once. Review fixes: add labeled trigger; drop target so the check attaches to the triggering event's head SHA. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * ci(til): keep expired topics eligible; enforce kebab slugs; tighten allowed-files - rejected-til-topics excludes PRs closed by the gh-aw expiry job (its own "automatically closed because it expired" comment): #92 and #101 were never rejected, only unreviewed. Verified on live data: [101,92,77,...] -> [77,...]. - Slugs must match ^[a-z0-9-]+$: proxy.js negotiates markdown only for /blog/[\w-]+, so #101's dotted slug would have lost its markdown twin. The fact-check pre-step fails any non-kebab post file deterministically; the writer prompts state the rule. - allowed-files content/blog/** -> content/blog/*.mdx (gh-aw's matcher is */** only, no character classes; this blocks subdirectories and non-MDX). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * ci(til): vendor fuzzy schedule (every 12h); compile now has 0 warnings Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * ci(til): open TIL PRs with the PAT so auto-merge triggers push workflows GitHub docs: events triggered by GITHUB_TOKEN do not create workflow runs, so bot-enabled auto-merges left main without push CI/CodeQL/Scorecard (none from 09-12 until #107, a human merge). The owner chose the existing PAT (GH_AW_CI_TRIGGER_TOKEN), scoped to create-pull-request only. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * ci(til): fact-check on nemotron-3-ultra (measured 4/8 vs 1/8 recall) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
sameerkhansf
deleted the
til/deepseek-v4-1-flash-review-2026-477fa46529d55c5f
branch
September 28, 2026 10:01
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
1 participant
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.
Caution
agentic threat detected
Threat detection flagged this output in warn mode. Manual review is REQUIRED before any follow-up automation.
Details
Potential security threats were detected in the agent output.
Review the workflow run logs for details.
Angle Chosen
This review focuses on DeepSeek-V4.1-Flash's architectural innovations in KV cache compression, particularly its Causal Encoder-Decoder (CED) architecture and Compressed Sparse Attention 2 (CSA2) mechanisms that achieve 890 bytes per token KV cache size.
Evidence Summary
Sources accessed:
Key claims verified:
Verification Checklist
Checklist of Claims Verified
Warning
Firewall blocked 1 domain
The following domain was blocked by the firewall during workflow execution:
us.aws.cdn.hf.coTo allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.