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

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

Build:
Build:
LAST BUILD BRANCH: chore/fix-all-style-findings
DEFAULT BRANCH: master
Ran 27 Jun 2026 12:00PM 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:52AM UTC coverage: 80.556% (-0.1%) from 80.658%
28288380668

Pull #152

github

wegamekinglc
Fix AAD tape Clear regression: clear adjointsMulti_ unconditionally

Copilot on PR #152 correctly flagged that the ForEachBlock refactor changed
Clear's semantics: the original native-branch Clear cleared adjointsMulti_
unconditionally, but ForEachBlock guarded it on Tape_::multi_ (a runtime flag
toggled by SetNumResultsForAAD). A tape that ran multi (populating
adjointsMulti_), then toggled to non-multi, would retain stale adjoints on
Clear instead of wiping them.

- Split ForEachBlock (guarded, for Mark/Rewind/RewindToMark) from
  ForEachBlockAll (unconditional, for Clear) so each function preserves its
  original adjointsMulti_ clearing semantics. Mark/Rewind/RewindToMark were
  already guarded in the original and stay guarded.
- Add a one-line comment at ForEachBlockAll explaining why adjointsMulti_ is
  cleared unconditionally (so the next reader/Copilot sees it is intentional).
- Add regression test TestClearEmptiesAdjointsMultiAfterMultiToNonMultiToggle
  (native-gated) that toggles multi->non-multi and asserts adjointsMulti_ is
  emptied by Clear. Verified it fails against the buggy ForEachBlock-only form
  (8 stale adjoints retained) and passes with the fix.
- Add // namespace Dal closing-brace comments to fdm1dmesher.hpp and
  concentrating1dmesher.cpp (style item, files were touched by this PR).

Native: 753/753. XAD: 752/752 (regression test compiled out).
Pull Request #152: chore: dedup math structural (AAD iterators, tape loop, Sobol seek, mesher epilogue)

24 of 26 new or added lines in 4 files covered. (92.31%)

122 existing lines in 8 files now uncovered.

6293 of 7812 relevant lines covered (80.56%)

3223299.49 hits per line

Uncovered Changes

Lines Coverage ∆ File
2
93.02
-1.57% dal-cpp/dal/math/aad/tape.cpp

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
6
84.85
1.75% dal-cpp/dal/math/aad/blocklist.hpp
2
96.3
-0.26% dal-cpp/dal/script/visitor/evaluator.hpp
1
57.35
0.64% dal-cpp/dal/math/random/sobol.cpp
1
96.61
-0.06% dal-cpp/dal/model/blackscholes.hpp
1
80.0
0.0% dal-cpp/dal/protocol/optiontype.hpp
Jobs
ID Job ID Ran Files Coverage
1 28288380668.1 27 Jun 2026 12:00PM UTC 218
80.56
GitHub Action Run
Source Files on build 28288380668
  • Tree
  • List 218
  • Changed 27
  • Source Changed 22
  • Coverage Changed 21
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #28288380668
  • Pull Request #152
  • 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