Sine property could have better representation

Not sure whether it is a bug...

The property "amplitude" of sine wave generator is the peak-to-peak value and the "base value" should better be the DC offset.

In German literature there is only one usage of "amplitude" (= peak value). It seems that in English literature amplitude can be peak value as well as peak-to-peak value.. Other simulations (spice, falstad) also support this property representation of sine waves.

It's a bit picky, but would support the general representation: u(t) = Û * sin (omega t) + U_DC

Yes, currently "Amplitude" refers to peak-to-peak amplitude.
And "Base voltage" refers to minimun voltage.

It's a bit picky, but would support the general representation:
u(t) = Û * sin (omega t) + UDC

Where:
Û is peak or "semi" amplitude.
UDC is reference or "midle" voltage.

Yes this is more convenient for a symmetric wave.

I think we could apply this to the new wave generator, the existing one is too simple and lacks a lot of features.

Just as an update.
Wavegen in 1.0.0 is using this representation.