Why does Rhs2116StimulusTrigger take "double" instead of "bool"?

[cjsha]

I'm wondering why Rhs2116StimulusTrigger takes "double" instead of "bool" as the type of its input observable.

For reference, Headstage64ElectricalStimulatorTrigger and Headstage64OpticalStimulatorTrigger do take "bool" as their input observable type. I prefer how the hs64 does it because the graph for triggering in the example workflow can be more simple.

[jonnew]

The trigger provided to an Rhs2116StimulusTrigger defines a delay in microseconds that will be applied prior to the defined stumulus.

bonsai-onix1/OpenEphys.Onix1/Rhs2116StimulusTrigger.cs

Line 37 in 3c6702b

device.WriteRegister(Rhs2116Trigger.TRIGGER, (uint)(delaySamples << 12 | 0x1));

This needs to be documented properly:

1. Explain the difference between per channel Delay property and this global sequence delay.
2. Update the RHS2116Trigger TRIGGER description with this information

[cjsha]

My follow-up question is: should hs64 trigger operators follow the same pattern? They have a delay property instead of taking in delay in their input sequences