fix: close current-head native review gaps - #57
Conversation
Signed-off-by: Seongho Bae <me@seonghobae.me>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus 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 |
877fe04
into
docs/product-technical-gap-baseline
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| println!( | ||
| "shape={CANDIDATES}x{DIMENSION}x{QUERIES} min_ms={:.3} mean_ms={mean:.3} p95_ms={:.3} max_ms={:.3} bit_mismatches={mismatched} approximate_top4_mismatches={top_four_mismatches} screened_top4_mismatches={screened_top_four_mismatches} maximum_ambiguity={maximum_ambiguity} max_abs_diff={maximum_absolute_difference:e}", | ||
| elapsed[0], elapsed[27], elapsed[29] | ||
| elapsed[0], elapsed[28], elapsed[29] |
There was a problem hiding this comment.
📝 Info: Nearest-rank index matches p95
Thirty sorted samples place nearest-rank p95 at rank 29. The zero-based selection elapsed[28] is correct.
Was this helpful? React with 👍 or 👎 to provide feedback.
| "pyproject.toml", | ||
| "SECURITY.md", | ||
| } | ||
| forbidden_prefixes = (".github/", ".git/") | ||
| forbidden_prefixes = (".github/", ".git/", "crates/") |
There was a problem hiding this comment.
📝 Info: Python-only boundary remains deterministic
OpenCode can edit broadly, but the pre-execution gate rejects crates/ and pyproject.toml. Validation uses the restored trusted native extension.
Was this helpful? React with 👍 or 👎 to provide feedback.
| def rank_authorized_batch_packed( | ||
| self, | ||
| model_identity: str, | ||
| query_vectors: list[list[float]], | ||
| packed_authorization: bytes, | ||
| ) -> list[ | ||
| tuple[ | ||
| tuple[str, str, str, str, str, str, int, int], | ||
| str, | ||
| str, | ||
| int, | ||
| str, | ||
| str, | ||
| list[tuple[str, str, float]], | ||
| ] | ||
| ]: ... |
There was a problem hiding this comment.
📝 Info: Collection contracts remain aligned
The native binding returns a list as declared. The public adapter converts it to an immutable tuple, preserving its separate contract.
Was this helpful? React with 👍 or 👎 to provide feedback.
1 participant
Summary
Validation
This remains a proof profile only and does not activate an accelerator or prove the consumer 20 ms concurrent SLO.