• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

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

Build:
Build:
LAST BUILD BRANCH: docs/full-review-and-perf-reflect
DEFAULT BRANCH: master
Ran 27 Jun 2026 11:10AM UTC
Jobs 1
Files 218
Run time 1min
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

27 Jun 2026 11:03AM UTC 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

Uncovered Changes

Lines Coverage ∆ File
9
19.67
2.77% dal-cpp/dal/script/visitor/debugger.hpp
5
89.52
8.41% dal-cpp/dal/script/simulation.hpp
3
59.24
2.18% dal-cpp/dal/script/visitor/domainproc.hpp

Coverage Regressions

Lines Coverage ∆ File
88
30.39
-1.35% dal-cpp/dal/script/visitor/compiler.hpp
14
56.9
2.8% dal-cpp/dal/storage/globals.cpp
9
28.57
-64.29% dal-cpp/dal/indice/indexparse.cpp
2
96.3
-0.26% dal-cpp/dal/script/visitor/evaluator.hpp
1
96.61
-0.06% dal-cpp/dal/model/blackscholes.hpp
1
80.0
0.0% dal-cpp/dal/protocol/optiontype.hpp
1
89.52
8.41% dal-cpp/dal/script/simulation.hpp
Jobs
ID Job ID Ran Files Coverage
1 28287308979.1 27 Jun 2026 11:10AM UTC 218
80.8
GitHub Action Run
Source Files on build 28287308979
  • Tree
  • List 218
  • Changed 22
  • Source Changed 19
  • Coverage Changed 16
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #28287308979
  • Pull Request #150
  • PR Base - master (#28286827538)
  • Delete
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc