docs(adr): 0006 -- daily strategies are evaluated on the pooled sample, never per asset (#822) - #824
Merged
Merged
Conversation
…e, never per asset (#822) Records the owner's 2026-09-26 decision on the floor question #427 left open, and formalizes #338's pooled unit of evaluation: n >= 100 pooled with the 5-products x 10-trades diversity floor, per-product statistics as diagnostics only, and every evaluation at that floor stated as descriptive with its power (n_eff ~39, ~20-point detectable edge). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
eaitbrahim
added
research
eaitbrahim
added a commit
that referenced
this pull request
Sep 26, 2026
…oled evaluation as policy (#828) MINOR: #826 changes what promotion decides (G2 and #338's pooled path judge in R; min_expectancy is an R threshold; a sample with no R is refused), and #825 adds report sections. No schema change since 0.17.0. What lands: #826 (#820) -- edge table, G2 and promotion in R; negative-risk flip fixed; pooled rows in exit-time order. #825 (#821) -- simulate buys DCA once per completed day (was 24x per cadence day), no forming-day look-ahead, DCA sleeve reported. #824 (#822) -- ADR 0006: daily strategies evaluated on the pooled sample. #827 (#823) -- experiment record: hourly turtle, 4,871 trades. #819 -- live per-asset ceiling config mirrored (0.50 -> 0.75). Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #822.
Adds
docs/decisions/0006-pooled-evaluation-of-daily-strategies.mdand its index entry.No code or config changes:
min_trades: 100keeps its value, and this record states its unit is the pool.🤖 Generated with Claude Code