docs: fix temporal topic and mathematical compute contracts - #617
Conversation
|
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 |
3e3b3ee
into
feat/dashboard-live-evidence
* feat(lineage): require TEPP criterion anchor for weights * fix(tepp): preserve exact cutoff precision * feat(dashboard): add evidence operations workspace * fix(tepp): enforce canonical anchor UUID * fix(frontend): keep merged OIDC imports minimal * feat: complete semantic dashboard evidence paths * fix(tepp): promote only exact accepted lineage vectors * Revert "fix(tepp): promote only exact accepted lineage vectors" This reverts commit bc182f1. * fix(tepp): promote exact accepted lineage vectors * fix: bind semantic retrieval to resolved model * fix: bind semantic retrieval to resolved model and page VOC * fix(ui): preserve settled calendar status semantics * fix(lineage): preserve focused isolation reason * fix: close semantic evidence review gaps * docs: refresh exact-head product gap baseline * fix(ui): preserve dashboard controls during reload * feat(dashboard): distinguish retryable analysis failures * fix(ui): isolate similar VOC pagination by post * docs: record exact-head dashboard acceptance evidence * fix(ui): keep failed-analysis guidance coherent * fix(ui): drop stale similar VOC pages after navigation * test: type similar VOC fetch fallback * feat(dashboard): cite linked evidence posts * fix(api): keep similar VOC query in one audited template * fix(ask): restrict evidence to semantic candidates * fix(api): bound similar VOC adjudication * docs: fix temporal topic compute contracts (#617) Co-authored-by: seonghobae <seonghobae@users.noreply.github.com> --------- Co-authored-by: seonghobae <seonghobae@users.noreply.github.com>
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| def _numerical_import_files() -> set[str]: | ||
| """Return production Python files importing a numerical owner package.""" | ||
|
|
||
| found: set[str] = set() | ||
| for base in (ROOT / "lineageweave", ROOT / "backend" / "app"): | ||
| for path in base.rglob("*.py"): | ||
| tree = ast.parse(path.read_text(encoding="utf-8"), filename=str(path)) | ||
| imports = { | ||
| alias.name.split(".", 1)[0] | ||
| for node in ast.walk(tree) | ||
| if isinstance(node, ast.Import) | ||
| for alias in node.names | ||
| } | ||
| imports.update( | ||
| node.module.split(".", 1)[0] | ||
| for node in ast.walk(tree) | ||
| if isinstance(node, ast.ImportFrom) and node.module | ||
| ) | ||
| if imports & NUMERICAL_OWNER_MODULES: | ||
| found.add(path.relative_to(ROOT).as_posix()) | ||
| return found |
There was a problem hiding this comment.
📝 Info: Guard freezes import surface, not local math
The guard in test_math_boundary_inventory.py flags only files importing {fast_mlsirm, numpy, rankweave, scipy, sklearn}. Local numerical code using stdlib math or hand-rolled arithmetic (e.g. embedding_client.py cosine, knowledge_graph.py RWR, backend/app/post_chat_ingestion.py, all named in the audit doc) passes undetected. The expected set matches today, so the test is green, but it does not prevent new pure-Python numerical implementations.
Was this helpful? React with 👍 or 👎 to provide feedback.
1 participant
Summary
Evidence
Stack