docs: S5 — refresh manager README + fix stale CIC line (#75)#82
Merged
Conversation
- views_postprocessing/unfao/managers/README.md: rewritten lean and accurate. The
old version (410 lines) predated the input-integrity guards, carried the wrong
PROD_FORECASTS_COLLECTION_ID, listed stale deps ("mapping utilities", polars), and
duplicated the root README. The new version documents the current stages incl. the
identity guard (C-25), coverage/excluded-cell guards (C-34/C-30), observed-range clip
(C-26) and structured provenance (C-15), the call-not-inherit pattern, and points to
the orientation doc + CICs instead of duplicating them.
- docs/CICs/UNFAOPostProcessorManager.md: fixed a stale factual line claiming the
runtime mapper "remains in the repo ... slated for removal" — it was removed (C-39).
The package/module map already lives in the root README (S3) + role_and_seams (S2), so
it is not triplicated here. Historical/decision docs (ADR-011, the falsification
campaign records, the cross-repo report) legitimately reference the old mapper and are
left intact. Docs-only; ruff clean, suite green.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.
Epic: #70 · S5 (#75) · depends on #73 ✅
unfao/managers/README.md— rewritten lean + accurate (was 410 lines, predated the input-integrity guards, wrong collection-ID, stale deps, duplicated the root README). Now documents the current stages incl. identity (C-25), coverage/excluded-cell (C-34/C-30), observed-range clip (C-26), provenance (C-15), the call-not-inherit pattern; links to the orientation doc + CICs instead of duplicating.CICs/UNFAOPostProcessorManager.md— fixed one stale line claiming the mapper "remains in the repo ... slated for removal" → it was removed (C-39).Package/module map already lives in README (S3) + role_and_seams (S2), not triplicated. Historical docs (ADR-011, falsification records, cross-repo report) reference the old mapper legitimately and are left intact (this informs S6's guardrail scope). All links resolve; docs-only; ruff clean, suite green.
Closes #75