Add \all param option

[capital-G]

Improves passing metadata to params within transpiler

DynGenDef(\x).compile(#{
            ~phase = (~phase + (~p.freq(200.0, spec: \freq)/~srate)).wrap(0.0, 1.0);
            ~sig = sin(~phase * 2 * pi + (~sig * ~p.fb(0.0, spec: [0.0, 2.0])));
            ~out0 = ~sig;
}).send,

Also allows to pass \all as param value in DynGen, which creates a named control for every param present in the code (w/ associated control spec/init value when using the transpiler). This allows to write

(
Ndef(\z, {
    var sig = DynGen.ar(
        numOutputs: 1,
        script: DynGenDef(\x).compile(#{
            ~phase = (~phase + (~p.freq(200.0, spec: \freq)/~srate)).wrap(0.0, 1.0);
            ~sig = sin(~phase * 2 * pi + (~sig * ~p.fb(0.0, spec: [0.0, 2.0])));
            ~out0 = ~sig;
        }).send,
        params: \all,
    );
    sig.dup * 0.2;
}).play.gui;
)

This also works w/o the transpiler as seen in the test testPlainDynGenParamExtract.

[Spacechild1]

If the second operand of and is an expression, it should be in a function. This way it is only evaluated if the first operand is true (= short circuiting):

if ((params.size == 1) and: { params[0] == \all }) { ... }

Note that the function will be inlined.

[Spacechild1]

all parameters of the script with as

all parameters of the script as

[Spacechild1]

Hmmm... I would expect this method to just return an Array of all parameter names or something. I definitely wouldn't expect it to have side effects (= creating NamedControls) for the containing Synth.

Maybe something like paramControls or makeControls could be clearer?

[Spacechild1]

This calls prExtractParameters everytime you create a new DynGen, although the parameters only change when the code changes. For this reason, wouldn't it make more sense to just call prRegisterParams in the code_ setter method (instead of in the send method)?

I guess we only want the parameters that are currently in the code. But this could be easily solved by keeping another array, e.g. prCurrentParams, that only contains the current parameters. We probably need this anyway when we expose the current parameters (and their specs) to the user.

[Spacechild1]

prRegisterParams calls prExtractParameters again!

Just remove the call to prExtractParameters here and modify prRegisterParams to store the extracted parameters in prCurrentParams.