FilterEvents has wrong log values

[Andrei Savici]

I tried to run the following script

Load(Filename=r'/SNS/ARCS/2012_2_18_CAL/data/ARCS_30489_event.nxs',OutputWorkspace='ARCS_30489_event',LoadMonitors='1')
GenerateEventsFilter(InputWorkspace='ARCS_30489_event',OutputWorkspace='f',InformationWorkspace='fi',UnitOfTime='Nanoseconds',LogName='CCR12Rot',MinimumLogValue='-50',MaximumLogValue='-20',LogValueInterval='1',FilterLogValueByChangingDirection='Decrease')
FilterEvents(InputWorkspace='ARCS_30489_event',OutputWorkspaceBaseName='filtered',InformationWorkspace='fi',SplitterWorkspace='f',FilterByPulseTime='1',GroupWorkspaces='1')

1. I expect that in workspace 1 to have events from CCR12Rot=-49 to -48. I get events from -49.5 to -48.5. The documentation needs to be changed. Or the algorithm should do as expected
2. The more problematic is the first (sometimes the last - depending on direction) workspace. In workspace 0 I expect to see data from -50.5 to -49.5 (in the current way of handling this), but the lower limit is -65.2. I am not sure if the events are filtered correctly, but I am sure that the log values are not

[Wenduo Zhou]

Cleaned codes and add some more debug output. Refs #7459.

Changeset: 14933e2d7921f904b9df217a9f557a106da81d64

[Wenduo Zhou]

Fixed bug and add debug log. Refs #7459.

Changeset: 5431ada6d60739d34fd629c5f2a647a73a275590

[Wenduo Zhou]

Documentation of GenerateEventsFilter is improved on how to specify the log value interval (http://www.mantidproject.org/GenerateEventsFilter)

[Andrei Savici]

Merge remote-tracking branch 'origin/bugfix/7459_filter_log'

[Andrei Savici]

It could still use some improvement in documentation

[Stuart Campbell]

This ticket has been transferred to github issue ​8304