Skip to content

Parity review needed - R NNS 7250bb627d12f6cdb1dbe60a8b8e50385e2a7c41 - #23

Merged
OVVO-Financial merged 2 commits into
mainfrom
parity-review-7250bb627d12f6cdb1dbe60a8b8e50385e2a7c41
Jun 14, 2026
Merged

Parity review needed - R NNS 7250bb627d12f6cdb1dbe60a8b8e50385e2a7c41#23
OVVO-Financial merged 2 commits into
mainfrom
parity-review-7250bb627d12f6cdb1dbe60a8b8e50385e2a7c41

Conversation

@OVVO-Financial

Copy link
Copy Markdown
Owner

Live-R parity for the changed R API needs human review.

Cause: Live R parity diverged: public Python behavior differs from live R at the recorded commit.

R commit: 7250bb627d12f6cdb1dbe60a8b8e50385e2a7c41
R version: 13.0

Reports committed on this branch:

  • sync/last_r_api_inspection.md
  • sync/last_r_api_plan.json
  • sync/last_live_r_parity_report.md

Current method (automated fixing agent deferred): a maintainer
reproduces the divergence against live R and applies a reviewed
parity fix to src/nns/**. Record the function, arguments, R output,
Python output, first divergent intermediate, and affected files.
Native differences route through NNS-core. A DESCRIPTION/version or
behavior change is a separate, reviewed cache regeneration. Do not
edit tests/_r_cache.json to force a pass.

@OVVO-Financial
OVVO-Financial merged commit 86c5199 into main Jun 14, 2026
4 checks passed
@OVVO-Financial
OVVO-Financial deleted the parity-review-7250bb627d12f6cdb1dbe60a8b8e50385e2a7c41 branch June 14, 2026 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant