this function should be removed. read_segment_times is not used anywhere and not tested.
it should also have use the time_as_index method.
I am +1 for removing it. Any objection?