adaptive-machine-learning / CapyMOA / 31136934490

80%

Build Type

push

github

Committed by hmgomes

Commit Message

fix(stream): store generator parameters so __str__ works

RandomRBFGenerator, RandomRBFGeneratorDrift, STAGGERGenerator,
SineGenerator, WaveformGenerator and WaveformGeneratorDrift build their
MOA CLI string from locals() and never assigned the arguments to self,
so __str__ read attributes that were never set and str()/print() raised
AttributeError. DriftStream.__str__ and DriftStream.describe() print
their concepts, so they failed for any drift stream built from one of
these six.

Assign the constructor arguments, as the other eight generators in the
module already do, and cover every public generator with a parameterized
str() test so a new one cannot regress in the same way.

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

Coverage Stats

38 of 38 new or added lines in 2 files covered. (100.0%)

33 existing lines in 1 file now uncovered.

8448 of 10728 relevant lines covered (78.75%)

0.79 hits per line