Fix issue that was causing full color elements to render lower intensities than they should in the preview.
Merge remote-tracking branch 'remotes/billlutton/master'

# By billlutton
# Via billlutton
* remotes/billlutton/master:
  More Instrumentation - no functional change. Added collection for the 3 phases of OutputController Update processing: -generating OutputCommands -Extracting commands to outputs -Controller processing commands
  Fix Exception (no sequence executor) when press Play after Execution Stop/Start. Essentially we just hook the Execution state changed event and free up our context on 'Closing' (it is going to be release anyway) and recreate it when Execution returns to 'Open' state.
  Fix Stack Overflow when re-grouping. Create ElementTree::GroupFromSelectedNodes was adding the new group to itself as a child because the AddSingleNodeWithPrompt changed the selected item in the element tree.  This fix simply saves/uses the original selection as the children to put under the new group.
  Fix Exception on Cancel from CreateGroup from selection. Also avoid firing OnElementsChanged on Cancels in a few other places.
  Start work on where the time goes in Device.Update...
Merge remote-tracking branch 'remotes/billlutton/master'

# By billlutton
# Via billlutton
* remotes/billlutton/master:
  Keep serial port Open execption from propagating (which cause problems later). Ebrady on doityourselfchristmas reported a problem cancelling out of the SSF.  When I loaded the config on my system (which doesn't have COM1, the serial port Open throws in the device Start.  This primed things to fail when the device was eventually Stopped.  Ebrady was cancelling out of SFF which reloads system config (stopping and stating all devices).  Stopping the device failed because Start never completed successfully.  This change keeps the Open exception from propagating, thus preserving the ability to Stop sussessfully - no more crash.
  Increase upper limit on UpdateInterval (so I can play Giz's new stuff). No functional change - just a change on the upper bound on the control.
  Avoid NPE if user enters bad name for elements.
  Fix defect where Execution would re-create/register it's counters and have them not be visible.   Basically, Instrumentation ion Add now just accepts the last counter registered with a given name.
  Switch underlying _values from Dictionary to OrderedDictionary. This will keep the values in the order they were created instead of letting them jump around in the list.
  Add controller name to the new generate/extract/device Instrumentation counters.  Without this some of them can get 'lost' by the Instrumentation display framework.
Merge remote-tracking branch 'remotes/jeffu231/master'