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

LAST BUILD BRANCH: docs/full-review-and-perf-reflect
DEFAULT BRANCH: master
Repo Added
Token RCHt5bHBHiesAZrXN9ctLtgBNi9sbYmV8 regen
Build 3548 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/api-dedup-safe
branch: chore/api-dedup-safe
CHANGE BRANCH
x
Sync Branches

coverage: 81.318% (+0.02%) from 81.299%
28294188053

push

github

wegamekinglc 
refactor: brace-init Build() + add CurveSolverOptions_ vocabulary type

Rewrite both CurveCalibrationSpecBuilder_::Build() and
CrossCurrencyCalibrationSpecBuilder_::Build() as aggregate brace-init of the
underlying Spec_ from the builder fields, replacing the member-by-member copy.
Eliminates the silent field-drop hazard when a Spec_ grows: the compiler now
enforces field-count agreement, and round-trip tests guard against order drift.

Add CurveSolverOptions_ to dal-public/src/curvespec.hpp as a public vocabulary
type centralizing the seven shared solver-knob names and the single-curve
defaults (tolerance_ 1e-8, initialGuess_ 0.05). The builders keep their flat
fields so dal-python member-pointer bindings (def_readwrite) and dal-excel
glue stay source-compatible; this is the non-breaking anchor for a future
breaking structural collapse. Sync-reminder comments added above both
builders' solver-field blocks.

Fully backward-compatible: no caller, binding, or Spec_ layout changes.

Tests: add CurveSpecTest.TestCurveSolverOptionsDefaults and per-builder
round-trip tests (CurveSpecTest.TestBuildRoundTripsEveryField,
XccyCalibrationTest.TestBuildRoundTripsEveryField) that populate every field
with distinct sentinels to catch any brace-init order/count mismatch.

Co-Authored-By: Claude <noreply@anthropic.com>

10 of 10 new or added lines in 3 files covered. (100.0%)

6281 of 7724 relevant lines covered (81.32%)

3253418.59 hits per line

Relevant lines Covered
7724 RELEVANT LINES 6281 COVERED LINES
3253418.59 HITS PER LINE
Source Files on chore/api-dedup-safe

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
28294188053 chore/api-dedup-safe refactor: brace-init Build() + add CurveSolverOptions_ vocabulary type Rewrite both CurveCalibrationSpecBuilder_::Build() and CrossCurrencyCalibrationSpecBuilder_::Build() as aggregate brace-init of the underlying Spec_ from the builder fields, r... push wegamekinglc github
81.32
28294196336 chore/api-dedup-safe Merge e31307fed into 08f0acccd Pull #172 web-flow github
81.32
See All Builds (2160)

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