Disable text boxes in Optical Stimulator when appropriate

- When the number of bursts or pulses is 1, the period and the inter-burst interval are undefined, so these text boxes are disabled

Author: bparks13

OpenEphys.Onix1.Design/Headstage64OpticalStimulatorOptions.cs

@@ -33,6 +33,7 @@ internal void UpdateSequenceParameters(ConfigureHeadstage64OpticalStimulator opt
             textBoxMaxCurrent.Text = opticalStimulator.MaxCurrent.ToString();
             textBoxPulseDuration.Text = opticalStimulator.PulseDuration.ToString();
             textBoxPulsePeriod.Text = opticalStimulator.PulsesPerSecond.ToString();
+            textBoxPulsePeriod.Enabled = opticalStimulator.PulsesPerBurst > 1;
 
             textBoxChannelOnePercent.Text = opticalStimulator.ChannelOneCurrent.ToString();
             trackBarChannelOnePercent.Value = (int)(opticalStimulator.ChannelOneCurrent * channelOneScalingFactor);
@@ -41,6 +42,7 @@ internal void UpdateSequenceParameters(ConfigureHeadstage64OpticalStimulator opt
 
             textBoxPulsesPerBurst.Text = opticalStimulator.PulsesPerBurst.ToString();
             textBoxInterBurstInterval.Text = opticalStimulator.InterBurstInterval.ToString();
+            textBoxInterBurstInterval.Enabled = opticalStimulator.BurstsPerTrain > 1;
             textBoxBurstsPerTrain.Text = opticalStimulator.BurstsPerTrain.ToString();
             textBoxDelay.Text = opticalStimulator.Delay.ToString();
         }

OpenEphys.Onix1.Design/Headstage64OpticalStimulatorSequenceDialog.cs

@@ -100,12 +100,22 @@ public Headstage64OpticalStimulatorSequenceDialog(ConfigureHeadstage64OpticalSti
                 { StimulusSequenceOptions.textBoxPulsesPerBurst,
                     new TextBoxBinding<uint>(
                         StimulusSequenceOptions.textBoxPulsesPerBurst,
-                        value => { OpticalStimulator.PulsesPerBurst = value; return OpticalStimulator.PulsesPerBurst; },
+                        value =>
+                        {
+                            OpticalStimulator.PulsesPerBurst = value;
+                            StimulusSequenceOptions.textBoxPulsePeriod.Enabled = OpticalStimulator.PulsesPerBurst > 1;
+                            return OpticalStimulator.PulsesPerBurst;
+                        },
                         uint.Parse) },
                 { StimulusSequenceOptions.textBoxBurstsPerTrain,
                     new TextBoxBinding<uint>(
                         StimulusSequenceOptions.textBoxBurstsPerTrain,
-                        value => { OpticalStimulator.BurstsPerTrain = value; return OpticalStimulator.BurstsPerTrain; },
+                        value =>
+                        {
+                            OpticalStimulator.BurstsPerTrain = value;
+                            StimulusSequenceOptions.textBoxInterBurstInterval.Enabled = OpticalStimulator.BurstsPerTrain > 1;
+                            return OpticalStimulator.BurstsPerTrain;
+                        },
                         uint.Parse) }
             };
 
@@ -281,9 +291,9 @@ static bool IsSequenceValid(ConfigureHeadstage64OpticalStimulator sequence, out
                 reason = "Maximum current is invalid.";
                 return false;
             }
-            else if (sequence.PulsesPerSecond <= sequence.PulseDuration)
+            else if (sequence.PulsesPerBurst > 1 && sequence.PulsesPerSecond <= sequence.PulseDuration)
             {
-                reason = "Pulse duration is too short compared to the pulse period.";
+                reason = "Pulse period is too short compared to the pulse duration.";
                 return false;
             }