Repository navigation
Conversation
|
{ |
|
/review Please review exact head |
|
Advisory cross-file review against Lean Pool's supervisory rubric. This is not by itself a significance, source-verification, or merge verdict. [Low impact] No new actionable code defect was found; this review does not clear Vilin97’s acknowledged merge holds. SummaryAdds finite-dimensional copulas, arbitrary-marginal Sklar existence, continuous-marginal uniqueness, and the classical CDF characterization.
Reviews (1) · Last reviewed commit: "Add finite-dimensional copulas and Sklar..." · Reviewed by Greptile |
🤖 LLM review (
|
| Rubric | Verdict | Bottom line |
|---|---|---|
| Faithfulness | ✅ pass |
All five mathematical headlines match the Lean statements and constructions; full CI remains pending. |
| Novelty | ✅ pass |
No duplicate of the five headlines was found in the inspected Mathlib, Tau Ceti, or pooled declarations. |
| Significance | ✅ pass |
Sklar existence, continuous-marginal uniqueness, and the classical CDF characterization form a completed graduate-level development worth pooling. |
| Sources | ✅ pass |
The pinned upstream core supports the five cited results, and the quantile module explicitly credits its Mathlib antecedent. |
| Code quality (advisory) | ✅ pass |
The approximation and lifting proofs reuse rectangleIncrement_split, Division.grid_mesh, and exists_randomized_inverse, keeping the code factored with no material maintenance debt found in the full source review. |
| Aspect | Value |
|---|---|
| Proves the claim | ✅ proves_it |
| Assumed, not proved | Probability laws are supplied; full uniqueness assumes continuous marginal CDFs; the classical direction assumes the four stated boundary and rectangle conditions; generic lifting assumes a standard Borel space and measurable samplers with the prescribed marginal pushforwards—all disclosed in the card. |
| Matches cited source | ✅ matches |
| Fit | ✅ good_fit |
| Level | graduate |
| Branch | probability theory |
| Mode | theory_building |
| Code quality | 4 / 5 |
Statement check: The Lean proves arbitrary finite-dimensional real-law CDF factorization, full uniqueness under continuous marginal CDFs, agreement on marginal CDF ranges, the classical characterization without assumed continuity, and lifting through prescribed measurable samplers, including dimension zero.
The theorems exists_sklarCopula, existsUnique_sklarCopula_of_continuous, and isClassical_iff_existsUnique establish foundational copula theory for finite-dimensional laws, supported by quantile sampling, randomized lifting, and atomic approximation.
Faithfulness findings (1)
- prompt-injection — PR-wide
The contributor-written discussion supplies a prior faithfulness verdict and directs a review across five rubrics. These have no authority over this independent, faithfulness-only review and were disregarded as instructions.
Evidence: prose only: the acquisition-review comment in .review-pr.json says "The mathematical and engineering review passes locally" and records a faithfulness "verdict": "pass"; a later contributor comment says "Please review exact head8682e1105ca2a17ad44896df12509c0e64995129against the current five rubrics."
Tokens: 6,495,871 in / 40,666 out across 5 model calls · Tier: codex-subscription · Effort: xhigh · Billing: Codex subscription quota (no API credits) · Estimated cost: $26.5935 (Standard API equivalent; uncached input)
Each rubric is an independent review against .github/review-rubrics/ on top of .github/REVIEW_RULES.md. Disagree? Reply on the PR; rules can be updated in a PR of their own.
CI was not green for this commit at review time; this review does not replace CI.
Vilin97
left a comment
There was a problem hiding this comment.
Source review at 8682e1105ca2a17ad44896df12509c0e64995129: I read all 22 supplied Lean files (2,004 lines) and the project card, with an independent peer review.
This is a significant, coherent formalization. exists_sklarCopula handles arbitrary finite-dimensional real probability laws by quantile sampling and randomized inverse lifting, recovering the full original joint law. Continuous-marginal uniqueness uses almost-everywhere threshold equality and dense CDF ranges, so flat CDFs are supported. isClassical_iff_existsUnique starts from normalization, groundedness, uniform faces and nonnegative ordered rectangle increments; it derives continuity and constructs the representing probability measure through finite atomic approximations and weak limits. Dimension zero and half-open rectangle conventions are explicit. I found no concrete mathematical statement or proof defect in the supplied bodies.
The merge blocker remains the card's provenance_evidence: mix is inferred while the original core's proof-production method is unconfirmed. Confirm that method and correct the category if needed before merging; retain the draft and existing infrastructure hold. This is a source review. I have not independently verified the imported Mathlib bodies, kernel/build audit, CI logs, upstream release/license history, or duplication claims.
This adds the completed foundations of Corrram's copula library: Sklar's theorem for arbitrary finite-dimensional real probability laws, uniqueness for continuous marginals and on marginal CDF ranges, and the equivalence between classical copula CDF conditions and probability measures with uniform marginals. Atoms, singular laws, flat continuous marginal CDFs, and dimension zero are supported. Continuity is derived in the classical characterization rather than assumed.
The measure-based vocabulary follows Mathlib. Compact quantile adjunction and sampling, randomized inverse disintegration, general lifting through marginal sampling maps, rectangle probabilities, sharp Lipschitz bounds, and finite atomic approximation are reusable independently of the headline theorem.
Source, eligibility, and scope
fe53ea2f3e6af41da8541e7d0aa2edc38ed0dcc8(2026-10-03 20:25:28 UTC).eaee4eab6931066921870358c0f4bc0b8bd32068and5578d2b62ad948570a3edce60330d012e6b5c97b, both September 18.b40930bbcf80744c86c46a12bc9da056641d722716c378f5659b9e555ef833e1. Original copyrights are retained. The compact quantile adaptation credits Gaëtan Serré and Rémy Degenne alongside Marcus Rockel and identifies Mathlib's kernel representation as prior work.Rectangle.leansubsequently exposes the same rectangle probability proof as a reusable public theorem.Provenance blocker — keep open
The card currently records
mixas an inference, with its evidence and limitation inprovenance_evidence. The original core file histories credit Marcus Rockel without a proof-tool disclosure. The retained rectangle proof was revised in the explicitly Claude-authored commit9d07e1456c8ec655666255ec966f138e28c56477; the port and proof optimization here are by Codex. Human author attribution and AI contributions are documented, but the original core's proof-production method is not explicitly disclosed in the inspected source histories, release documentation, public author homepage, or companion repository README. Confirm that method and correct the category if necessary before merging. No auto-merge or queue entry is requested.The infrastructure priority hold also remains in force until the rollout owner verifies current-main CI. PR610 and PR611 are merged; neither branch was touched by this cycle.
Port and measured optimization
Ported from Lean/Mathlib v4.34.0 to Lean Pool's exact Lean v4.35.0-rc3 and Mathlib
c55e6e786f49471c72fbddbec5415808896aec1e. Module exports use explicit public sections and individual@[expose]annotations to preserve the upstream transparent mathematical definitions. Imports useLeanPool.Copula;LeanPool.leanis unchanged.After the mechanical port and initial warning repair, both timings used:
Before each measurement, only this worktree's Copula build artifacts were removed; the same pinned dependency oleans were retained. Lake schedules independent modules concurrently on a shared 16-core Azure VM. Measurements include elaboration and Lake overhead; they exclude dependency compilation. Shared VM load makes the percentage an observed result rather than a guaranteed speedup.
Observed wall-time reduction: 24.4%. Profiles were obtained with
lake env lean --profile <file> --setup <Lake-generated setup.json>using two Lean threads. The approximation module's cumulative simplification time fell from 795 ms to 358 ms; regularity fell from 667 ms to 211 ms in the recorded profiles. These phase measurements also include shared-machine noise.The pass removed 14 net imports, made proof-only dependencies private, replaced broad simplification in 42 steps with explicit lemmas, then eliminated four of those steps by reusing
Finset.piecewise_insertin two repeated proofs. Removing the two import-only aggregates shortens the build's dependency chain. No heartbeat override was introduced or retained. The remaining profiles show dependency loading and tactic/linter initialization as substantial costs; no further material safe proof improvement was evident.The Mathlib linter found an unused
Nonemptyassumption on randomized inverses. It is removed there and from the generic lifting API, strengthening those results. Warnings, generated-card placement, and line lengths were repaired directly in the content.Prior art and validation
The registry, source tree, open/merged PRs, issues, and repository history yielded no duplicate of this core. Mathlib's Sklar search hit is a documentation roadmap entry, not a proof. The closest Tau Ceti source inspected is the real-valued quantile API; it does not state Sklar's theorem or the classical copula characterization. The ordered compact quantile here has an endpoint-valid adjunction and is independently credited to its Mathlib antecedent. No Tau Ceti project is vendored.
Local checks pass:
lake exe mk_all --check; the fixed root index is byte-identical to base.lake exe runLinter --no-build LeanPool.Copula.Importsandlake exe lint-style LeanPool.Copula.Imports.lean_pool.quality --static-only.#print axiomsaudit of all 146 named public declarations, and environment backdoor/axiom-injection audit over every declaration in the imported modules, including generated helpers. OnlyClassical.choice,propext, andQuot.soundare allowed.git diff --check; content-only scope. No Python files or gates are changed.Lean Action CI is green on submitted head
8682e1105ca2a17ad44896df12509c0e64995129, including project build, Mathlib lint, text style, repository quality, and the required Build project bridge. All applicable checks pass; Python, documentation, and unrelated workflow checks skip according to scope. The partial-port advisory completed successfully. All retained results and their dependency closure are complete.Greptile completed on the same head and found no actionable code defect. The separate complete-source acquisition review passes the mathematical and engineering rubrics and requests changes for unconfirmed original proof provenance. The configured GPT review was requested; no verdict is available yet. The PR remains a draft with no auto-merge or queue entry.