Repository navigation
C2 — Retire views_postprocessing/reconciliation/ (moved to views_frames_reconcile) #62
Description
Activity
- addedstoryA single reviewable unit of an epicA single reviewable unit of an epicimplementationCode implementation workCode implementation workblockedBlocked on a dependency or decisionBlocked on a dependency or decision
on Jun 26, 2026 Done in PR #63 (merged to
development2026-06-26, merge commit c9e3840). Deletedviews_postprocessing/reconciliation/(6 modules) + 6 tests + 2 parity fixtures + 2 gen scripts + theReconciliationModuleCIC; migration doc → pointer to views-frames; dropped the stale CIC-index entry (review fix).ruff+pytestgreen (74 passed / 43 xfailed); CI (check-branch, Analyze, CodeQL) green.Deletion-safety verified on
origin/developmentpre-merge: no unguarded importer ofviews_postprocessing.reconciliationanywhere — C1 (views-models#191) had already repointedreconciler_factorytoviews_frames_reconcile(PR #202); only guardedpytest.importorskiptests remain (skip when absent). (Closesdidn't auto-fire: PR targeteddevelopment, not the default branch.)✅ Unblocked — the reconciliation cutover landed (Decision K)
The reconciler's live home is now `views_frames_reconcile` (views-frames v1.7.0 sibling). views-pipeline-core PR #217 merged to `development` (`6427b9d`, 2026-06-26): pipeline-core reconciles via its `Reconciler` port, and views-models (`reconciler_factory`) constructs + injects `views_frames_reconcile.ReconciliationModule`. Nothing imports vpp's `reconciliation/` copy — it's a stranded way-station (vpp was only ever a parity oracle; FAO delivery doesn't reconcile).
Note: this is Decision K, not the earlier C plan — but it doesn't change C2's premise. vpp's reconciliation copy is dead either way, so this issue proceeds as written: delete `views_postprocessing/reconciliation/` + its tests/fixtures/scripts, and repoint `docs/CICs/ReconciliationModule.md` at `views_frames_reconcile`. (The earlier "blocked on C1 / views-models repoint" gate is satisfied — views-models already imports `views_frames_reconcile`.)
- added a commit that references this issue
on Aug 1, 2026
Cross-repo cutover for views-frames Epic 11 (views-platform/views-frames#131).
blockedon C1 (views-models repoint). Closes #31.Problem
The reconciler has moved to the
views_frames_reconcilesibling in the views-frames distribution and consumers are repointed (C1). views-postprocessing was only ever a way-station — FAO delivery doesn't use reconciliation. Delete our copy so the repo screams its real job (partner delivery).Work
views_postprocessing/reconciliation/(proportional.py, grouping.py, frames.py, validation.py, module.py, init.py).tests/test_reconciliation_*.py, fixturestests/fixtures/reconciliation_*.npz, andscripts/gen_reconciliation_*.py.docs/CICs/ReconciliationModule.md+ the migration notes to point at views-frames; move register C-37/C-38 tracking to views-frames.Acceptance criteria
PYTHONPATH=. pytest -qgreen;ruffclean.Dependencies
Files
delete
views_postprocessing/reconciliation/,tests/test_reconciliation_*.py,tests/fixtures/reconciliation_*.npz,scripts/gen_reconciliation_*.py; updatedocs/CICs/ReconciliationModule.md.