wegamekinglc / Derivatives-Algorithms-Lib

81%

master: 81%

LAST BUILD ON BRANCH docs/migrate-script-comments

branch: docs/migrate-script-comments

CHANGE BRANCH

x

Reset

Sync Branches

• docs/migrate-script-comments
• china_2025
• chore/add-doc-writer-to-orchestrator-team
• chore/api-dedup-safe
• chore/ban-volatile-mutable
• chore/curve-dedup-step1
• chore/curve-dedup-step2
• chore/curve-dedup-step3
• chore/curve-hoist-helpers
• chore/dedup-black-bachelier
• chore/dedup-cg-bcg
• chore/dedup-curve
• chore/dedup-dal-public
• chore/dedup-math-structural
• chore/dedup-model-indice
• chore/dedup-script
• chore/fix-infra-bugs
• chore/fix-io-protocol-indice-bugs
• chore/fix-math-bugs
• chore/fix-model-bugs
• chore/fix-script-bugs
• chore/phase7-curve-trim
• chore/remove-co-authored-by-from-guides
• chore/simplify-comments-phase1
• chore/simplify-comments-phase2
• chore/simplify-comments-phase3
• chore/simplify-comments-phase4
• chore/simplify-comments-phase5
• chore/simplify-comments-phase6
• chore/simplify-comments-phase8
• chore/simplify-comments-phase9
• chore/trim-curve-comments
• chore/trim-dupire-comments
• chore/trim-script-comments
• chore/trim-small-residuals
• ci/expand-ci-to-public-subprojects
• ci/matrix-workflow
• copilot/add-test-coverage-analysis
• copilot/analyze-test-coverage
• copilot/analyze-test-coverage-again
• copilot/design-web-ui-derivatives-portfolio-management
• copilot/enhance-yield-curve-framework
• copilot/explore-codebase-implementation-plan
• copilot/explore-codebase-yield-curve-modeling
• copilot/override-function-gradient-calibration
• copilot/refactor-script-engine
• dependabot/github_actions/actions/checkout-3
• dependabot/github_actions/actions/checkout-4
• dependabot/github_actions/actions/checkout-5
• dependabot/github_actions/actions/checkout-6
• dependabot/github_actions/actions/checkout-6.0.2
• dependabot/github_actions/actions/checkout-6.0.3
• dependabot/github_actions/actions/checkout-7.0.0
• dependabot/github_actions/astral-sh/setup-uv-8.2.0
• dependabot/github_actions/astral-sh/setup-uv-caf0cab7a618c569241d31dcd442f54681755d39
• dependabot/npm_and_yarn/webui/frontend/multi-f57e1e291f
• dependabot/npm_and_yarn/webui/frontend/vite-8.0.16
• dependabot/uv/dal-web/backend/starlette-1.3.1
• docs/add-black-scholes-methodology
• docs/add-dupire-methodology
• docs/add-matrix-methodology
• docs/add-no-dup-and-comment-style-rules
• docs/add-pde-methodology
• docs/add-quadrature-methodology
• docs/add-random-methodology
• docs/claude-config-sync
• docs/copilot-instructions-guidance
• docs/design-api-shape-dedup
• docs/fix-documentation-consistency
• docs/full-quality-pass
• docs/full-review-and-perf-reflect
• docs/improve-readme-formatting
• docs/markdown-table-alignment-and-cleanup
• docs/migrate-curve-comments
• docs/migrate-small-residuals
• docs/ptirds-curve-extension-plan
• docs/restructure-and-webui-rename
• docs/simplify-comments-phase0
• docs/simplify-comments-phase7
• experimental
• feature/aad-methodology-doc
• feature/adept-aad-backend
• feature/adept-expr-tape-tests
• feature/agent-team-design
• feature/agents-tdd-worktree
• feature/analytic-jacobian-backend-abstraction
• feature/analytic-jacobian-rename
• feature/code-style-fixes
• feature/codipack-aad-backend
• feature/codipack-polish
• feature/comment-migration-docs
• feature/coverage-build-flag
• feature/currency-unit-tests
• feature/curve-jacobian-default-analytic
• feature/curve-jacobian-mode-flag
• feature/dal-doc-writer-agent
• feature/dal-pr-review-agent
• feature/dal-python-uv-modernization
• feature/dal-refactor-docs
• feature/dal-refactor-full
• feature/dal-refactor-phase0-phase1
• feature/dal-refactor-phase2
• feature/dal-refactor-phase3
• feature/dal-refactor-phase4
• feature/dal-refactor-phase5
• feature/dal-unit-test-write-agent
• feature/dal-web-playwright-e2e
• feature/docs-interp-logdf-calibration-examples
• feature/docs-table-style-guidance
• feature/docs-webui-startup
• feature/enforce-worktree-dal-dev-workflow
• feature/euribor3m-curve-calibration-example
• feature/expose-curve-calibration-to-python-and-excel
• feature/harden-dal-pr-review-agent
• feature/joint-curve-base-layering
• feature/log-linear-interp-and-refactor
• feature/multi-curve-joint-calibration
• feature/normal-expectation-quadrature
• feature/piecewise-linear-curve-tests
• feature/ptirds-log-discount-curve
• feature/ptirds-single-currency-curve
• feature/pybind11-migration
• feature/pybind11-submodule
• feature/remove-precompute-self-param
• feature/rename-skills-dal-prefix
• feature/reorg-miscs-to-subfolders
• feature/script-engine-test-coverage
• feature/serialization-tests-and-skills
• feature/sync-agent-doc-guidance
• feature/webui-redesign-industrial-terminal
• feature/windows-ci-workflow
• feature/ycinstrument-polymorphic-calibration
• feature/yield-curve-calibration
• feature/yield-curve-jacobian-example
• fix/adept-cmake-definitions
• fix/arrayn-dimension-validation
• fix/calibration-build-constant
• fix/cg-shadow-vector-alloc
• fix/code-style-param-naming
• fix/doc-writer-methodology-refs
• fix/machinist-cmake-compiler-id
• fix/msvc-ndarray-test-parsing
• fix/nodiscard-warnings-script-tests
• fix/priority-8-test-cosmetics-and-guidance
• fix/readme-build-badge
• fix/underdetermined-guidance-and-xad-default
• fix/upgrade-vite-plugin-react-for-vite8
• fix/xccy-calibration-budget
• fix/xccy-calibration-coverage
• fix/yield-curve-inverse-jacobian-units
• master
• old_virtual_dispatch
• perf/bench-infrastructure
• perf/enable-fp-contract-all-compilers
• perf/p1-tape-rewind-p5-zeroadjoints
• perf/p2-sparse-jacobian
• perf/p3-pde-decomp-caching
• perf/p4-sobol-no-polish
• perf/p6-p7-p8-kernel-opts
• perf/p7-p8-fused-sweeps
• refactor/rename-calibratedyieldcurve-to-curveblock
• refs/heads/master
• refs/pull/5/merge
• revert/ptirds-101
• use_codi
• wegamekinglc-perf-improvements
• worktree-agent-a344d3a27476b1053

Build # 28290090484

Build Type

push

github

Committed by wegamekinglc

Commit Message

docs: migrate script-engine residual rationale comments into methodology

Move two design-rationale comment blocks from dal-cpp/dal/script/ into the
Visitor Machinery and Parser sections of docs/methodology/script_engine.md,
so the prose lives where the no-large-comments rule requires it and the
source can be trimmed to one-line pointers in a follow-up wave.

- Lexer subsection under Parser and AST: Tokenize is the single tokenization
  primitive shared by the preprocessor (definition front-end) and parser
  (payoff back-end); housed in its own translation unit to keep the two
  halves decoupled.
- Visit-Trait Dispatch subsection under Visitor Machinery: the
  IsVisitorConst / HasConstVisit_ / HasNonConstVisit_ traits resolve
  const-ness and Visit-overload presence at compile time via Pack and
  SFINAE, so dispatch carries no virtual overhead beyond the double-
  dispatch table. Const-Correctness subsection now names these traits as
  the mechanism behind its compile-time enforcement claim.

Doc-only; no source edited. Current-state voice, no source line numbers,
no historical narrative. The preprocessor.hpp "built for extension"
rationale was already covered verbatim by the existing Preprocessing
Pipeline section and needs no additional doc change.

Coverage Stats

6300 of 7763 relevant lines covered (81.15%)

3236781.62 hits per line