Skip to content

Regenerate R parity cache for NNS 13.2 - #141

Merged
OVVO-Financial merged 4 commits into
mainfrom
automation/r-cache-nns-13.2
Aug 18, 2026
Merged

Regenerate R parity cache for NNS 13.2#141
OVVO-Financial merged 4 commits into
mainfrom
automation/r-cache-nns-13.2

Conversation

@OVVO-Financial

@OVVO-Financial OVVO-Financial commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Lands the R parity cache regenerated from the NNS 13.2 source of truth, brought up to date with current main.

The cache content here was produced by the Regenerate R parity cache workflow (run #55, 2026-08-03), which auto-dispatched when the 13.2 archives were pushed upstream. Every step of that run passed, including live regeneration and cache-only verification, but its pull request was never merged, so main continued to carry the 2026-07-18 cache. This branch is that same regenerated cache, rebased forward.

Provenance

  • R repository: OVVO-Financial/NNS
  • R commit: ec17f6979d3480767c6d047ce3fa50b95b2b7d29 ("NNS 13.2 Beta", 2026-08-03)
  • NNS version: 13.2
  • Source package: NNS_13.2.tar.gz
  • Windows binary verified: NNS_13.2.zip

sync/nns_source.json moves r_commit from bb878ad9c3e8282c30eaf47d219a5630db1b1a9d to ec17f6979d3480767c6d047ce3fa50b95b2b7d29, recording the archive-building commit as behavioral-truth provenance. The vendored native snapshot (tools/NNS, r_src_tree_hash, tarball) and the NNS-core commit are unchanged; re-syncing the native snapshot remains a separate NNS-core update.

Cache changes by function

function before after added removed changed
NNS.reg 295 295 0 0 9
dy.dx.numeric 2 2 0 0 2
NNS.ANOVA.custom 15 15 0 0 1
NNS.reg.factor_dimred 3 3 0 0 1

13 entries changed in total. No entries were added or removed, and no shard files were created or deleted — all 58 shards (2314 entries) report nns_version 13.2.

Also included: RUF036 lint fix

CI installs ruff unpinned, and ruff 0.16.x promoted RUF036 ("None not at the end of the type union") to stable. The rule fires on three pre-existing unions, which failed the Run ruff step on main and on every open pull request — main's own runs #844, #847 and #848 were all red for this reason, and because ruff precedes them in the job, mypy and Build distributions were being skipped entirely.

Commit 087e79d reorders the members so None comes last:

  • src/nns/pm_matrix.pyTarget
  • tests/_r.pyJsonValue and RValue

Union member order carries no semantics in Python typing, so the annotations are unchanged in meaning. This is bundled here rather than split out because the parity cache cannot go green on its own while the base is red, and two of the three unions live in tests/_r.py, already part of this workflow's own add-paths surface.

Verification

The full CI pipeline was run locally against this tree, in CI's order, with the package built from source:

step result
ruff check . (0.16.3, matching CI) All checks passed
mypy Success — 48 source files
native import smoke test 1.25
pytest tests/invariants 350 passed, 1 skipped
NNS_R_CACHE_ONLY=1 pytest tests/parity 1727 passed, 12 skipped
pytest tests/property 59 passed
pytest tests/plotting 33 passed
pytest tests/docs/test_vignette_examples.py 9 passed
python -m build sdist + wheel built

The 12 parity skips are the live-R-only practical examples (no Rscript available) plus one test retired as an obsolete contract; both are expected in cache-only runs.

🤖 Generated with Claude Code

https://claude.ai/code/session_01RCeNnGqf75qssMttPCeLaz

OVVO-Financial and others added 4 commits August 3, 2026 18:49
ruff is installed unpinned in native-backend-ci, and ruff 0.16.x promoted
RUF036 ("None not at the end of the type union") to stable. The rule now
fires on three pre-existing unions, failing the lint step on main and on
every open pull request.

Reorder the members so None comes last. Union member order carries no
semantics in Python typing, so the annotations are unchanged in meaning;
ruff 0.16.3 and mypy both pass clean afterward.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RCeNnGqf75qssMttPCeLaz
@OVVO-Financial
OVVO-Financial merged commit 41f3d7d into main Aug 18, 2026
5 checks passed
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.

2 participants