It defaults to 500 bins.
This allows event data to be plotted without rebinning first.
This is a change from before when it would all be in one bin.
It does not change the underlying data that remains in events.

Includes unit test, and doc changes.

Newer boost bind seems to be causing
compiler warnings on Windows. It's no
longer required now bind is in the STL.
Refs #16578

Signed-off-by: Jose Borreguero <borreguero@gmail.com>

Casting to a long64 does not match the types defined as acceptable for
table workspaces. TableColumn.cpp indicates that int64_t should be used
instead.

There was a check for certain file endings, but it did not account for the original length of the path.

When a Group workspace was loaded into the dialog for the algorithm, it
would cause an unhandled exception due to a null pointer. There is now a
check to ensure that a group workspace is not used.

Keeping with integer arithmetic during the conversion
means that the time resolution cannot go below 1us and
artifacts in the data are produced.
Added tests for data/sample logs for small bin widths.

Martyn Gigg authored 5 years ago and Gigg, Martyn Anthony committed 5 years ago

The header uses ptr_fun which is deleted in C++17
so our code won't compile and it can be replaced
with boost filesystem and a few lines of code.

Confirmed with SNS (W. Zhou) that this is no
longer required. Usage stats confirm it has not been
used since 3.13 so no deprecation is required.
Refs #27059

Nick Draper authored 5 years ago and Gigg, Martyn Anthony committed 5 years ago

As that is what the code in exec expects anyway