were not well tested. The latter one (Bursts.recompute_index_reduce) had only a stub implementation which was wrong.
This PR add tests for these 3 methods, complete the implementation of Bursts.recompute_index_reduce and fixes a bug in Bursts.recompute_times (found through unit tests) which was affecting the burst search when compact=True (the default is False) was used. Additionally there are some cleanups and docstring improvements.
The three methods:
were not well tested. The latter one (Bursts.recompute_index_reduce) had only a stub implementation which was wrong.
This PR add tests for these 3 methods, complete the implementation of Bursts.recompute_index_reduce and fixes a bug in Bursts.recompute_times (found through unit tests) which was affecting the burst search when
compact=True
(the default is False) was used. Additionally there are some cleanups and docstring improvements.