fix: preserve saturated lazy ROC call clocks#64
Merged
Conversation
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
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.
Outcome
Preserve TradingView's observed saturated lazy-call clock for the two direct
ta.roc(close, 3)sites in QuantByBoji without changing ROC globally.close[3]behavior for first call and gap-one/two casesThe classifier is deliberately fail-closed. It excludes UDF,
request.security,or, ternary, aliased/dynamic source, non-literal/other length, named/dynamic argument, nested control-flow, reassignment, expression-statement, and user-shadowedcloseshapes.TradingView evidence
The long sites were pinned by E2/E4/E5. The separately reviewed one-shot S2 short-site oracle is public in pinescript-scrapper PR #29:
+0.038753340977, uniquely saturated q11.025336599509/1.139529453306Exact target gate
136/136/136/13600 / 0 / 0e39e7a9fc89c4198e9b2a6b799acd05851d760c001154c4dbfa2071683729bbcef65c3fc2d10c1c8754f43a0d2a5912ba4617988b6ca951a10cdfb6d0cd50eeeThe rebase onto public codegen
228fe316preserved stable patch-ID7de31e6b. Versus the pre-rebase generated Quant output, the only additional base delta is the already-published argument-preservingstrategy.close_all("END of DAY", ...); the engine tape remains byte-identical.Regression gates
0, 1,348 collected with 2 existing skips2dc25ec89711324c8cd028d78680a39ed7c9b389, corpusaeff79e4af1052528010e1b9cded5dd0cc677021404 ok + 1 no-trades + 6 transpile + 1 existing run error)344 Excellent / 50 Strong / 3 Moderate / 5 WeakModerate -> Excellent,1 UP / 0 DOWN03367c5edc5f962311ed691172e75d44e883f604afebd8ac17d0913f265f6845dd5708c05880094833dde07769ba6df1e21c0b41b9d7fdb7358c44f02a5bfc9dgit diff --check: PASS