Skip to content
This repository was archived by the owner on Feb 18, 2026. It is now read-only.

Commit c52b44b

Browse files
committed
chore(fmt)
1 parent 11c2b0d commit c52b44b

1 file changed

Lines changed: 14 additions & 12 deletions

File tree

_done/20250209-extraction-complete.md

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -87,18 +87,18 @@ source file does. Moving `diskann.c` to `_research/` is safe.
8787

8888
**Test coverage gap analysis — what's already covered vs what's missing:**
8989

90-
| Scenario | Existing Coverage | Gap? |
91-
| ------------------------------- | -------------------------------------------------------------------- | ---------------------------- |
92-
| Search empty index | `test_search_empty_index` | No |
93-
| Insert wrong dimensions | `test_insert_dimension_mismatch` | No |
94-
| Delete nonexistent | `test_delete_nonexistent_id` | No |
95-
| Drop → open fails | `test_drop_index_removes_shadow_table` + `test_open_index_not_found` | No |
96-
| Insert → delete → search | `test_insert_delete_search` + `test_integration_delete_at_scale` | **Covered** |
97-
| Recall (brute-force comparison) | `test_insert_recall` + `test_integration_recall_128d` | **Covered** |
98-
| Reopen persistence | `test_integration_reopen_persistence` | **Covered** |
99-
| Clear → reinsert → search | `test_integration_clear_reinsert` | **Covered** |
100-
| Higher-dim recall | `test_integration_recall_128d` | **Covered** |
101-
| Larger delete+search | `test_integration_delete_at_scale` | **Covered** |
90+
| Scenario | Existing Coverage | Gap? |
91+
| ------------------------------- | -------------------------------------------------------------------- | ----------- |
92+
| Search empty index | `test_search_empty_index` | No |
93+
| Insert wrong dimensions | `test_insert_dimension_mismatch` | No |
94+
| Delete nonexistent | `test_delete_nonexistent_id` | No |
95+
| Drop → open fails | `test_drop_index_removes_shadow_table` + `test_open_index_not_found` | No |
96+
| Insert → delete → search | `test_insert_delete_search` + `test_integration_delete_at_scale` | **Covered** |
97+
| Recall (brute-force comparison) | `test_insert_recall` + `test_integration_recall_128d` | **Covered** |
98+
| Reopen persistence | `test_integration_reopen_persistence` | **Covered** |
99+
| Clear → reinsert → search | `test_integration_clear_reinsert` | **Covered** |
100+
| Higher-dim recall | `test_integration_recall_128d` | **Covered** |
101+
| Larger delete+search | `test_integration_delete_at_scale` | **Covered** |
102102

103103
**Recall target of 95% may be aggressive with random start.** The existing
104104
`test_insert_recall` only requires 60% on 50 vectors/3D. For 200+ vectors at 128D with
@@ -225,8 +225,10 @@ are separate projects.
225225
4. Mark all remaining phases complete and move this TPP to `_done/`.
226226

227227
**Files created this session:**
228+
228229
- `tests/c/test_integration.c` — 4 integration tests (reopen, clear+reinsert, 128D recall, delete at scale)
229230

230231
**Files modified this session:**
232+
231233
- `tests/c/test_runner.c` — added 4 extern declarations + RUN_TEST calls
232234
- `_todo/20250209-extraction-complete.md` — this file (validated and updated throughout)

0 commit comments

Comments
 (0)