03 March 2014
John Clyne, Alan, Scott
@mention a person to add them as an attendee and they will be notified.
Review Params changes proposal
The corresponding Doxygen to look at is the following:
-Everything in the Command class (this is the support for the Undo/Redo Queue)
-The related methods on ControlExecutive (Undo, Redo, GetCommandText, GetCurrentCommand)
-Plus, the relevant methods in Params, ParamsBase, including
SetValidationMode, GetValidationMode, Validate
Params::CaptureSetLong, Params::CaptureSetDouble, Params::CaptureSetString, Params::CaptureSetStringVec,
and the corresponding methods on ParamsBase.
If you would like to see how these methods are being used, check the various SetValue methods in RegionParams, ViewpointParams, AnimationParams, ArrowParams, Box, and Viewpoint classes. These methods are also used in the gui EventRouter classes (RegionEventRouter, ViewpointEventRouter, AnimationEventRouter, and ArrowEventRouter) however the EventRouters are not currently in Doxygen.
We decided to simplify the Validation as follows:
We also decided that there will be no "dirty bits" to indicate when there has been a change in a Params. Instead, renderers will need to check whether there has been a change to the state before rendering. Renderers can also make use of the ParamNode DirtyFlag. The DirtyFlag can be used to enable a renderer to check if a particular value has been modified, When a DirtyFlag has been registered, the dirtyFlag gets set whenever the associated element is modified, and it can be tested and cleared by the renderer.