[N-02] Gaussian math audit

[ericnordelo]

Summary by CodeRabbit

• Documentation
  • Clarified documentation for related distribution evaluators.
  • Updated comments describing safeguards used during central-domain calculations.
  • No changes to functionality, interfaces, or behavior.

[coderabbitai]

📝 Walkthrough

Walkthrough

Documentation-only updates revise evaluator-structure and overshoot-guard comments in cdf.move. Imports, logic, exported entities, and interfaces are unchanged.

Changes

cdf Documentation

Layer / File(s)	Summary
Clarify evaluator and guard comments math/fixed_point/sources/internal/cdf.move	Updates sibling-evaluator documentation and reframes the overshoot guard as defense-in-depth without changing its condition or clamping logic.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• OpenZeppelin/contracts-sui#431: Updates the central-domain rational evaluation and related documentation in cdf.move.

Suggested reviewers: 0xneshi, bidzyyys, immrsd

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	No description was provided, so the required issue reference, change summary, and checklist are missing.	Add the required template sections: issue reference, a brief change summary, and the PR checklist items for tests, documentation, and changelog.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title matches the PR's main theme of a Gaussian math audit and is reasonably specific.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/gaussian-audit-N-02

---

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.54%. Comparing base (f880455) to head (9bd72b4).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #475      +/-   ##
==========================================
+ Coverage   96.45%   96.54%   +0.08%     
==========================================
  Files          34       34              
  Lines        3414     3414              
  Branches      799      798       -1     
==========================================
+ Hits         3293     3296       +3     
  Misses         73       73              
+ Partials       48       45       -3     

Flag	Coverage Δ
contracts/access	65.46% <ø> (ø)
contracts/allowance	52.40% <ø> (ø)
contracts/finance	26.66% <ø> (ø)
contracts/timelock	54.32% <ø> (ø)
contracts/utils	44.09% <ø> (ø)
math/core	86.97% <ø> (+0.22%)	⬆️
math/fixed_point	63.70% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[bidzyyys]

LGTM