wegamekinglc / Derivatives-Algorithms-Lib
81%
master: 81%

LAST BUILD BRANCH: perf/p7-p8-fused-sweeps
DEFAULT BRANCH: master
Repo Added
Token RCHt5bHBHiesAZrXN9ctLtgBNi9sbYmV8 regen
Build 3545 Last
Files 221
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH chore/dedup-script
branch: chore/dedup-script
CHANGE BRANCH
x
Sync Branches

coverage: 80.804% (+0.1%) from 80.658%
28287308979

Pull #150

github

wegamekinglc 
De-duplicate script visitor and simulation paths

Three byte-identical refactorings under dal-cpp/dal/script/, no behavior change:

- debugger.hpp: extract DebugComp(CompNode_, const char*) private helper; the
  NodeEqual_/NodeSup_/NodeSupEqual_ visitors become one-line delegations. The
  only intra-branch difference was the label string ("EQUALZERO"/"GTZERO"/
  "GTEQUALZERO"); everything else (isDiscrete_/eps_/lb_/rb_ formatting) was
  identical.

- domainproc.hpp: introduce VisitBinary(Node_&, OP_) template mirroring the
  existing pattern in constprocessor.hpp and evaluator.hpp; the four
  Visit(NodeAdd_/NodeSub_/NodeMulti_/NodeDiv_) bodies collapse to one-line
  delegations. NodePow/NodeMax/NodeMin and unaries left untouched (genuinely
  distinct stack semantics).

- simulation.hpp (MCSimulation<AAD::Number_>): the per-path recording loop was
  duplicated in the compiled/else branches, differing only by evaluator type
  (EvalState_ vs FuzzyEvaluator_) and the product call (EvaluateCompiled vs
  Evaluate). Factor the loop into a runPaths(evaluator, evaluate) generic
  lambda; the verbatim params risk-accumulation loop is hoisted out of the
  branch (it has no evaluator dependency); the const-vars risk loop is hoisted
  into an accumulateConstVarRisks(constVarVals) lambda called per branch with
  the branch-local evaluator. AAD recording order (RewindToMark per path,
  PropagateToMark per path, PropagateMarkToStart before reading adjoints) is
  preserved exactly.

Verified: dal_cpp_tests built and run on adept and xad backends.
Pull Request #150: chore: de-duplicate script visitors and AAD simulation path

19 of 36 new or added lines in 3 files covered. (52.78%)

116 existing lines in 7 files now uncovered.

6293 of 7788 relevant lines covered (80.8%)

3235545.49 hits per line

Relevant lines Covered
7788 RELEVANT LINES 6293 COVERED LINES
3235545.49 HITS PER LINE
Source Files on chore/dedup-script

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
28287308979 chore/dedup-script De-duplicate script visitor and simulation paths Three byte-identical refactorings under dal-cpp/dal/script/, no behavior change: - debugger.hpp: extract DebugComp(CompNode_, const char*) private helper; the NodeEqual_/NodeSup_/NodeSupEqual_ v... Pull #150 wegamekinglc github
80.8
See All Builds (2158)

Badge your Repo: Derivatives-Algorithms-Lib

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh