Skip to content

Clear the set-member signature cache per pathway (deltasignal specs/065) - #115

Merged
adamjohnwright merged 1 commit into
mainfrom
sig-cache-per-pathway
Oct 16, 2026
Merged

adamjohnwright merged 1 commit into
mainfrom
sig-cache-per-pathway

Conversation

@adamjohnwright

Copy link
Copy Markdown
Contributor

prime_entity_caches cleared every per-pathway entity cache except variant_emission._sig_cache, so in a catalog build (one process, 92 pathways) a set member's signature could be the one computed from an earlier pathway's prefetch-limited caches: the network depended on build order. Week review finding 12.

  • Fix: clear _sig_cache in prime_entity_caches.
  • Test: test_priming_a_pathway_clears_set_member_signatures (fails without the fix).
  • Measurement pre-registered in deltasignal specs/065; variant build and bench to follow. Draft until measured.

🤖 Generated with Claude Code

prime_entity_caches cleared every per-pathway entity cache except
variant_emission._sig_cache, so in a multi-pathway build a member's
signature could come from an earlier pathway's prefetch-limited caches
and the network depended on build order. Week review finding 12.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@adamjohnwright
adamjohnwright marked this pull request as ready for review October 16, 2026 14:22
@adamjohnwright

Copy link
Copy Markdown
Contributor Author

Measured in deltasignal specs/065: all 92 networks identical in content, no predicted class changes. Ready to merge.

@adamjohnwright
adamjohnwright merged commit cbbd694 into main Oct 16, 2026
4 checks passed
@adamjohnwright
adamjohnwright deleted the sig-cache-per-pathway branch October 16, 2026 14:30
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